[SSSD] [PATCHES] Another batch of fixes for bugs found by Coverity
Simo Sorce
ssorce at redhat.com
Fri Dec 17 13:12:03 UTC 2010
On Fri, 17 Dec 2010 07:25:43 -0500
Stephen Gallagher <sgallagh at redhat.com> wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> On 12/16/2010 04:25 PM, Simo Sorce wrote:
> >> Patch 0002: https://fedorahosted.org/sssd/ticket/713
> >
> > NACK, failing to set non-blocking should probably be a fatal
> > failure, as any operation would get us stuck on the socket blocking
> > everything else (signal processing and all).
>
>
> Excellent point. I've modified the patch to reflect this. (And also
> fixed a related issue in close-on-exec that I found nearby)
>
> Please re-review.
Looks good, but I am not sure if failing to set close-on-exec should be
a fatal failure. I don't think it can ever happen, but even if it does,
we leak a file at most.
OTOH, given it will never happen it is probably a good idea to return
an error if it ever happens so we can inspect what is going on as
someone will notice.
So ACK.
Simo.
--
Simo Sorce * Red Hat, Inc * New York
More information about the sssd-devel
mailing list