On Wed, 2013-08-14 at 17:57 +0200, Lukas Slebodnik wrote:
On (14/08/13 11:46), Simo Sorce wrote:
>On Wed, 2013-08-14 at 13:50 +0200, Lukas Slebodnik wrote:
>> ehlo,
>>
>> Struct sss_auth_token became opaque in commit
>> 9acfb09f7969a69f58bd45c856b01700541853ca.
>> All ocasions of "struct sss_auth_token" was replaced with pointer to
this
>> structure, but proper initialization of auth_tokens was missing
>> in struct authtok_conv.
>>
>> Patch is attached.
>
>NACK
>You call free() on a talloced structure.
>You want to call talloc_free() there.
of course
>
>Otherwise LGTM.
>
>Simo.
>
Thank you for review. It was copy&paste problem.
Updated patch is attached.
Ack,
Simo.
--
Simo Sorce * Red Hat, Inc * New York