[SSSD] [PATCH] fix early free of sdap_handle

Simo Sorce ssorce at redhat.com
Wed Apr 14 14:41:42 UTC 2010


On Wed, 14 Apr 2010 18:21:50 +0400
Eugene Indenbom <eindenbom at gmail.com> wrote:

> Yes, there is nothing ideal. But there is one substantial difference 
> between workaround and solution:
>      - Workaround fixes problem by changing __other__ (client) code
> in the way that avoids triggering a bug in the faulty code.
>      - Solution fixes the buggy code so it starts to work as expected 
> without putting a burden on client code of following unnecessary 
> restrictions.

I agree, I am sending a new patch on top of the last one to move all
frees in the connection _recv() code.

This way we are guaranteed that it can't be called within a callback.
This means no unnecessary restriction are necessary.

Simo.

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



More information about the sssd-devel mailing list