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.