On Fri, 2009-10-16 at 10:57 -0400, Stephen Gallagher wrote:
> There was a double-free here. I removed the free()s within
> setup_db, because upon returning to sss_init_tools(), if
> ret != EOK, the context is freed as well.

ACK

we should *never* free a context that is passed to us, unless we are a
"termination" function. Only the allocating function should free
contexts.

Simo.

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

_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to