[SSSD] [PATCH] Separate return paths for success and failure in sdap_nested_group_check_cache

Stephen Gallagher sgallagh at redhat.com
Wed May 25 14:42:06 UTC 2011


On Wed, 2011-05-25 at 15:25 +0200, Jakub Hrozek wrote:
> This patch is related to https://fedorahosted.org/sssd/ticket/878
> 
> Although with how our code currently works, there is no way the
> uninitialized variable could be returned to user, we might in future add
> new calls that return arbitrary errno values. In that case the check for
> value of ret would not be sufficient.
> 
> The attached patch adds a separate fail: label used for failure exits
> from the function.

Ack and pushed to master.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20110525/8e38c725/attachment.sig>


More information about the sssd-devel mailing list