4bd34d
From 0379411547792a58b3d36c9928354072b5f6cabf Mon Sep 17 00:00:00 2001
4bd34d
From: Robbie Harwood <rharwood@redhat.com>
4bd34d
Date: Mon, 15 Apr 2019 19:56:50 -0400
4bd34d
Subject: [PATCH] Handle gss_import_cred() failure when importing gssx creds
4bd34d
4bd34d
Otherwise, we might attempt to set options on a non-existent handle,
4bd34d
leading to a segfault.
4bd34d
4bd34d
Signed-off-by: Robbie Harwood <rharwood@redhat.com>
4bd34d
Reviewed-by: Simo Sorce <simo@redhat.com>
4bd34d
Merges: #244
4bd34d
(cherry picked from commit 84cf88f6c6cfeb8e3fd2c26ed0fe9fe5bf3810d2)
4bd34d
---
4bd34d
 src/gp_export.c | 4 ++++
4bd34d
 1 file changed, 4 insertions(+)
4bd34d
4bd34d
diff --git a/src/gp_export.c b/src/gp_export.c
4bd34d
index 403e339..7ad8037 100644
4bd34d
--- a/src/gp_export.c
4bd34d
+++ b/src/gp_export.c
4bd34d
@@ -476,6 +476,10 @@ uint32_t gp_import_gssx_cred(uint32_t *min, struct gp_call_ctx *gpcall,
4bd34d
     }
4bd34d
 
4bd34d
     ret_maj = gss_import_cred(&ret_min, &token, out);
4bd34d
+    if (ret_maj) {
4bd34d
+        GPDEBUG("gss_import_cred failed when importing gssx cred\n");
4bd34d
+        goto done;
4bd34d
+    }
4bd34d
 
4bd34d
     /* check if there is any client option we need to set on credentials */
4bd34d
     gp_set_cred_options(cred, *out);