Guys,
This is great, thank you for finding it. But this will really mess up my merges. Can it be undone? Also I have a ticket open to explicitly return to this part of code and sort things out with valgrind. Thanks for finding the answer and cause of the problem, it will make the fixes simple but would it possible next time to ask me first or at least wait for my comment or ack before pushing? I have not fell off the planet at least yet...
Thanks Dmitri
Stephen Gallagher wrote:
On 09/16/2009 11:53 AM, Stephen Gallagher wrote:
On 09/16/2009 11:52 AM, Jakub Hrozek wrote:
On 09/16/2009 05:35 PM, Martin Nagy wrote:
Obviously, valgrind was producing warnings precisely because of
the bug
that Jakub's patch fixes. Therefore it was right to complain. I think the right thing to do right now is to remove all the explicit initializations and the comment about valgrind complaints.
That is absolutely correct, I even verified this with valgrind. New patch attached. Jakub
Ack
sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel
Pushed to master. _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel
_______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel