[Freeipa-devel] [PATCH 0046] Separate RR data parsing from LDAP connections

Petr Spacek pspacek at redhat.com
Thu Aug 16 09:08:57 UTC 2012


On 08/16/2012 11:01 AM, Adam Tkac wrote:
> On Wed, Aug 15, 2012 at 04:04:26PM +0200, Petr Spacek wrote:
>> On 08/15/2012 03:31 PM, Adam Tkac wrote:
>>> On Wed, Aug 01, 2012 at 04:19:11PM +0200, Petr Spacek wrote:
>>>> Hello,
>>>>
>>>> this patch finishes LDAP connection vs. LDAP result separation.
>>>>
>>>> It is first step necessary for:
>>>> https://fedorahosted.org/bind-dyndb-ldap/ticket/68
>>>> Avoid manual connection management outside ldap_query()
>>>>
>>>> It should prevent deadlocks in future.
>>>>
>>>> Petr^2 Spacek
>>>
>>>
>
> ...
>
>>> I recommend to use ZERO_PTR(ldap_qresult) instead of many "= NULL" assignments.
>>> It's safer and used in other *_create functions.
>>
>> Amended patch is attached.
>
> Ack
>
Pushed to master:
https://fedorahosted.org/bind-dyndb-ldap/changeset/ce547f03a86fbbcdb2db0629da615f04a35579b8

-- 
Petr^2 Spacek




More information about the Freeipa-devel mailing list