[SSSD] [PATCH] File descriptor leak in nss responder.

Simo Sorce simo at redhat.com
Thu Mar 7 13:42:16 UTC 2013


On Thu, 2013-03-07 at 13:44 +0100, Michal Židek wrote:
> Ok :)
> So I removed the unlink() from destructor (and the sss_mc_set_recycled
> with it). I added new dret variable to catch return value from
> unlink()
> in the done: part of sss_mc_create_file(), so that we do not overwrite
> the original error code.
> 
> Also the mc_ctx->fd should be set to -1 if the file construction in
> sss_mc_create_file (so that we do not try to close garbage file
> descriptor in the destructor). So I added this too.
> 
> New patches are attached.

I still see the unlink in the destructor ...

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




More information about the sssd-devel mailing list