[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