[Freeipa-devel] [PATCHES][SSSD] Fix two segfaults related to config file reloading

Stephen Gallagher sgallagh at redhat.com
Fri Jun 19 15:35:26 UTC 2009


0001: This segfault was caused by only talloc_steal()ing the first entry
in a linked list. The other entries were freed, causing a segfault the
next time they were accessed. I've added a memory context for the whole
linked list that can be passed to the new config in order to bring along
all of the entries in the list.

0002: There is a race condition (that is fairly involved and will be
dealt with separately) where signal_service_reload will attempt to
signal a child process that does not yet have an active sbus channel. In
that case, we'll just skip signaling it (since it won't get its initial
configuration until it's up anyway).
-- 
Stephen Gallagher
RHCE 804006346421761

Looking to carve out IT costs?
www.redhat.com/carveoutcosts/
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 0001-Fix-segfault-in-update_monitor_config.patch
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20090619/3a67a68e/attachment.ksh>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 0002-Protect-against-segfault-in-service_signal_reload.patch
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20090619/3a67a68e/attachment-0001.ksh>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 260 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20090619/3a67a68e/attachment.sig>


More information about the Freeipa-devel mailing list