|author||Chuck Lever <firstname.lastname@example.org>||2013-03-23 08:11:28 -0400|
|committer||Steve Dickson <email@example.com>||2013-03-25 10:09:10 -0400|
gssd: Clean up gssd_setup_krb5_user_gss_ccache()
Remove a contradictory portion of the block comment documenting gssd_find_existing_krb5_ccache(). This should have been removed by commit 289ad31e, which reversed the meaning of the function's return values. Note that, in user space, typically errno's are positive. But here we follow the kernel convention of using negative values to return error codes. Make the documenting comments explicit about the sign of an error return -- it will never be positive in the case of an error. And a nit: At the last return statement in gssd_setup_krb5_user_gss_ccache(), "err" always contains zero, as far as I can tell. Make it explicit (to human readers) that when execution reaches this point, gssd_setup_krb5_user_gss_ccache() is going to return "success." Reviewed-by: Jeff Layton <firstname.lastname@example.org> Signed-off-by: Chuck Lever <email@example.com> Signed-off-by: Steve Dickson <firstname.lastname@example.org>
1 files changed, 8 insertions, 8 deletions
diff --git a/utils/gssd/krb5_util.c b/utils/gssd/krb5_util.c
index 8178ae7..4df840e 100644
@@ -169,13 +169,13 @@ select_krb5_ccache(const struct dirent *d)
* Look in directory "dirname" for files that look like they
- * are Kerberos Credential Cache files for a given UID. Return
- * non-zero and the dirent pointer for the entry most likely to be
- * what we want. Otherwise, return zero and no dirent pointer.
- * The caller is responsible for freeing the dirent if one is returned.
+ * are Kerberos Credential Cache files for a given UID.
- * Returns 0 if a valid-looking entry was found and a non-zero error
- * code otherwise.
+ * Returns 0 if a valid-looking entry is found. "*cctype" is
+ * set to the name of the cache type. A pointer to the dirent
+ * is planted in "*d". Caller must free "*d" with free(3).
+ * Otherwise, a negative errno is returned.
gssd_find_existing_krb5_ccache(uid_t uid, char *dirname,
@@ -1042,7 +1042,7 @@ err_cache:
* given only a UID. We really need more information, but we
* do the best we can.
- * Returns 0 if a ccache was found, and a non-zero error code otherwise.
+ * Returns 0 if a ccache was found, or a negative errno otherwise.
gssd_setup_krb5_user_gss_ccache(uid_t uid, char *servername, char *dirpattern)
@@ -1087,7 +1087,7 @@ gssd_setup_krb5_user_gss_ccache(uid_t uid, char *servername, char *dirpattern)
printerr(2, "using %s as credentials cache for client with "
"uid %u for server %s\n", buf, uid, servername);
- return err;
+ return 0;