[Freeipa-devel] [PATCH 0017] dirsrv crash on segment add if suffix does not exist

thierry bordaz tbordaz at redhat.com
Tue Jun 30 13:37:35 UTC 2015


On 06/30/2015 12:07 PM, Ludwig Krispenz wrote:
> added verification for issue reported in ticket 5088 and sanity checks 
> requested in review for patch 0014
>
>
Hello,

The fix looks good except those sanity settings:

  * In ipa_topo_post_del, tsegm needs to be NULL initialized
  * In ipa_topo_check_segment_is_valid or ipa_topo_pre_add, I think
    *errtxt should be initialized to NULL

thanks
thierry

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20150630/53e04241/attachment.htm>


More information about the Freeipa-devel mailing list