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.

Otherwise LGTM.

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York

_______________________________________________
sssd-devel mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to