[SSSD] [PATCH] Fix a bad free in async_resolv.c

Simo Sorce ssorce at redhat.com
Fri Nov 6 17:07:49 UTC 2009


On Fri, 2009-11-06 at 17:51 +0100, Martin Nagy wrote:
> On Fri, 2009-11-06 at 08:12 -0500, Simo Sorce wrote:
> > On Fri, 2009-11-06 at 13:37 +0100, Martin Nagy wrote:
> > > Simple one-liner.
> > 
> > Nack, just remove the else free completely.
> > It will be freed anyway as soon as you return from the function in
> 99%
> > of the cases as the first thing after the _recv() is usually to free
> the
> > request.
> > 
> > Simo.
> 
> As you wish. But in that case, I'll do the same for the two other
> functions for consistency. New patch attached.

Ack.

Simo.

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




More information about the sssd-devel mailing list