[SSSD] [PATCH] NSS: Fix memory leak in sss_setnetgrent

Jakub Hrozek jhrozek at redhat.com
Tue Dec 10 11:04:31 UTC 2013


On Tue, Dec 10, 2013 at 11:38:06AM +0100, Jakub Hrozek wrote:
> On Wed, Dec 04, 2013 at 03:46:11PM +0100, Lukas Slebodnik wrote:
> > ehlo,
> > 
> > struct nss_cmd_ctx was not released in function nss_cmd_setnetgrent_done
> > and it wasn't used in the other function, because getnetgrent creates its
> > own nss_cmd_ctx context. struct nss_cmd_ctx was released after closing client
> > because it was allocated under client context. Memory leak is apparent with
> > long living clients.
> > 
> > Simple patch is attached.
> > 
> > LS
> 
> ACK.
> 
> I tested by using Lukas's reproducer from rhbz#1038098 along with a
> talloc_report() on the responder context. The memory consumption remained
> the same with the patch.

Pushed to master, sssd-1-11 and sssd-1-9



More information about the sssd-devel mailing list