From: Jeremy Filizetti <jeremy.filizetti@gmail.com>
Date: Mon, 14 Dec 2015 04:52:46 +0000 (-0500)
Subject: LU-3289 gss: Return GSS major and minor status
X-Git-Tag: 2.8.55~73
X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=cc45b429c83c0506169dbdcdbe7d917900981664;p=fs%2Flustre-release.git

LU-3289 gss: Return GSS major and minor status

Return the major and minor status codes back from the server side
userspace component (lsvcgssd) instead of defaulting to
GSS_S_FAILURE for the major and 0 for the minor.

Signed-off-by: Jeremy Filizetti <jeremy.filizetti@gmail.com>
Change-Id: I14d1a4b4ab6fb982e3c6c9c062d585865aa77ca0
Reviewed-on: http://review.whamcloud.com/17599
Tested-by: Jenkins
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Sebastien Buisson <sbuisson@ddn.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
---

diff --git a/lustre/ptlrpc/gss/gss_svc_upcall.c b/lustre/ptlrpc/gss/gss_svc_upcall.c
index 8944ef8..047b71a 100644
--- a/lustre/ptlrpc/gss/gss_svc_upcall.c
+++ b/lustre/ptlrpc/gss/gss_svc_upcall.c
@@ -182,7 +182,7 @@ static void rsi_free(struct rsi *rsi)
         rawobj_free(&rsi->out_token);
 }
 
-/* See handle_req() userspace for where the upcall data is read */
+/* See handle_channel_req() userspace for where the upcall data is read */
 static void rsi_request(struct cache_detail *cd,
                         struct cache_head *h,
                         char **bpp, int *blen)
@@ -1017,8 +1017,11 @@ cache_check:
         if (!rsci) {
                 CERROR("authentication failed\n");
 
-                if (!gss_pack_err_notify(req, GSS_S_FAILURE, 0))
-                        rc = SECSVC_COMPLETE;
+		/* gss mechanism returned major and minor code so we return
+		 * those in error message */
+		if (!gss_pack_err_notify(req, rsip->major_status,
+					 rsip->minor_status))
+			rc = SECSVC_COMPLETE;
 
                 GOTO(out, rc);
         } else {