[SSSD] [PATCH] Autofs Coverity issues

Jan Zelený jzeleny at redhat.com
Wed Mar 7 14:20:48 UTC 2012


> [PATCH 1/2] Autofs: operate on contents of double-pointer, not address
> This is a bad one, luckily the code is defensive later on and detects
> that we have processed all the records and quits correctly.
> 
> [PATCH 2/2] Only free returned values on success
> The pointers are set to NULL when the map is not found or the code
> iterates over all the maps, but it is good manners to only free the data
> when the caller is sure he received any.

Ack to both

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/20120307/c2337714/attachment.sig>


More information about the sssd-devel mailing list