Learn more about these different git repos.
Other Git URLs
It seems that my experience with valgrind was affected by some specific code paths that might have had issues and confused valgrind (and thus me).[[BR]] It seems that I came to a wrong conclusion that valgrind does not count calloc() and memset() as a valid initializers for allocated memory. As a result in multiple places I started to explicitly initialize the members of the structures.[[BR]] In some cases it makes sense but usually does not. The patch submitted recently uses calloc() and does not have any issues with valgrind. So this bug calls for review of the places where different context and config structures are allocated and refactoring of the initialization code.
Fields changed
owner: dpal => jhrozek
Fixed with 8a01230
fixedin: => 0.6.0 resolution: => fixed status: new => closed
rhbz: => 0
Metadata Update from @dpal: - Issue assigned to jhrozek - Issue set to the milestone: SSSD 0.6.0
SSSD is moving from Pagure to Github. This means that new issues and pull requests will be accepted only in SSSD's github repository.
This issue has been cloned to Github and is available here: - https://github.com/SSSD/sssd/issues/1203
If you want to receive further updates on the issue, please navigate to the github issue and click on subscribe button.
subscribe
Thank you for understanding. We apologize for all inconvenience.
Login to comment on this ticket.