[SSSD] [PATCH] [INI] Memory leak in case of empty value

Dmitri Pal dpal at redhat.com
Thu Jun 10 14:23:51 UTC 2010


Stephen Gallagher wrote:
> On 06/10/2010 09:44 AM, Stephen Gallagher wrote:
>   
>> On 06/09/2010 04:30 PM, Dmitri Pal wrote:
>>     
>>> Stephen Gallagher wrote:
>>>       
>>>> On 06/09/2010 04:11 PM, Dmitri Pal wrote:
>>>>
>>>>         
>>>>> Fixes #513.
>>>>>
>>>>> Thanks to Steve for helping to find it!
>>>>> Could not see it myself.
>>>>>
>>>>>
>>>>>           
>>>> Nack. Please address warning:
>>>>
>>>>      CC     ini_config_ut.o
>>>> ../../../common/ini/ini_config_ut.c: In function ‘get_test’:
>>>> ../../../common/ini/ini_config_ut.c:1011: warning: format ‘%ld’ expects
>>>> type ‘long int’, but argument 2 has type ‘char *’
>>>>
>>>>
>>>>
>>>>         
>>> Oops... Thanks for catching!
>>> New patch attached.
>>>       
>> Ack.
>>
>>     
>
> Addendum: this patch only applies cleanly on sssd-1-2. Please rebase and 
> resubmit a version of this patch that applies cleanly on master.
>
>   
Yes. I know. But I will do it later.

-- 
Thank you,
Dmitri Pal

Engineering Manager IPA project,
Red Hat Inc.


-------------------------------
Looking to carve out IT costs?
www.redhat.com/carveoutcosts/




More information about the sssd-devel mailing list