From 79cd7ef9ab52de98bffcbd1c8db6103ff985a263 Mon Sep 17 00:00:00 2001
From: Julien Chaffraix <julien.chaffraix@gmail.com>
Date: Sat, 2 Oct 2010 00:39:51 -0700
Subject: [PATCH] krb5-gssapi: Made the function always return a value.

kr5_auth missed a final 'return' statement. This is not an error in
gcc but can lead to potential bugs.
---
 lib/krb5.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/lib/krb5.c b/lib/krb5.c
index 5658bf428a..842bda17bb 100644
--- a/lib/krb5.c
+++ b/lib/krb5.c
@@ -170,7 +170,7 @@ krb5_encode(void *app_data, const void *from, int length, int level, void **to,
 static int
 krb5_auth(void *app_data, struct connectdata *conn)
 {
-  int ret;
+  int ret = AUTH_OK;
   char *p;
   const char *host = conn->host.name;
   ssize_t nread;
@@ -316,6 +316,7 @@ krb5_auth(void *app_data, struct connectdata *conn)
 
     service = srv_host;
   }
+  return ret;
 }
 
 static void krb5_end(void *app_data)
-- 
GitLab