[Freeipa-devel] [PATCH 101] Fix error handling in ldap_entry_create()

[Petr Spacek]

Hello,

apparently I was very tired yesterday ... Cleaned version of the patch is 
attached.

Petr^2 Spacek

On 11/21/2012 05:04 PM, Petr Spacek wrote:
> Hello,
>
> fixed fix is attached. Clang found bug in the fix but I didn't notice that
> because of other warnings ...
>
> On 11/21/2012 04:04 PM, Petr Spacek wrote:
>> Hello,
>>
>> I noticed this problem during investigation of dead code found by Clang.
>>
>>      Fix error handling in ldap_entry_create().
>>
>>      Jump to cleanup section after first memory allocation created memory leak
>>      which crashed BIND on reload.
>>
>>      Missing return value check after ldap_get_dn() call can lead to crash.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bind-dyndb-ldap-pspacek-0101-3-Fix-error-handling-in-ldap_entry_create.patch
Type: text/x-patch
Size: 1768 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20121122/564f42ed/attachment.bin>