#161 ELAPI Revew places in code where the contexts are allocated
Closed: Fixed None Opened 14 years ago by dpal.

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

Fields changed

rhbz: => 0

Metadata Update from @dpal:
- Issue assigned to jhrozek
- Issue set to the milestone: SSSD 0.6.0

7 years ago

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.

Thank you for understanding. We apologize for all inconvenience.

Login to comment on this ticket.

Metadata