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

Tomas Babej tbabej at redhat.com
Wed Jul 1 10:29:49 UTC 2015



On 07/01/2015 12:11 PM, thierry bordaz wrote:
> On 06/30/2015 04:50 PM, Ludwig Krispenz wrote:
>> new patch attached
>>
>> On 06/30/2015 03:37 PM, thierry bordaz wrote:
>>> 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
>>>
>>
> ACK
> 
> thanks
> thierry
> 
> 

Pushed to master: 5b76df4e7335c723f3fb14ef809e4d71e53509c9




More information about the Freeipa-devel mailing list