[SSSD] [PATCH] Silence Coverity warning in the autofs test tool

Jan Zelený jzeleny at redhat.com
Mon Mar 26 09:17:37 UTC 2012


> I think Coverity is being too noisy here, because only if
> _sss_getautomntent_r() succeeds, then key and value are allocated. This
> fix should hopefully explain the code to Coverity, too :-)

I have a question regarding the part of code where you delete exit() calls and 
replace them with jumps to end: label. There are two places like this left in 
the code, is that ok? IMO they could also lead to proper cleanup instead of 
just exiting.

Thanks
Jan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20120326/2929516a/attachment.sig>


More information about the sssd-devel mailing list