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