From 513a8fee6e60a7a369664aca9ba15789672783bd Mon Sep 17 00:00:00 2001 From: Christof Hanke Date: Wed, 28 Aug 2013 14:51:13 +0200 Subject: [PATCH] Linux Keyring error handling honour the returncode of key_instantiate_and_link() to avoid having non-working pagsh without an error. Reviewed-on: http://gerrit.openafs.org/10179 Tested-by: BuildBot Reviewed-by: Marc Dionne Reviewed-by: Michael Meffie Reviewed-by: Mark Vitale Reviewed-by: Derrick Brashear (cherry picked from commit c7f5a212025172307e85e09ada03333e5e288cc3) Change-Id: Ib5809e36ae8e87190206d75286bbeb23ace34b25 Reviewed-on: http://gerrit.openafs.org/10255 Reviewed-by: Marc Dionne Tested-by: BuildBot Reviewed-by: Stephan Wiesand --- src/afs/LINUX/osi_groups.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/afs/LINUX/osi_groups.c b/src/afs/LINUX/osi_groups.c index f5609ec0e..9916d2b42 100644 --- a/src/afs/LINUX/osi_groups.c +++ b/src/afs/LINUX/osi_groups.c @@ -253,7 +253,7 @@ setpag(cred_t **cr, afs_uint32 pagvalue, afs_uint32 *newpag, key = afs_linux_key_alloc(&key_type_afs_pag, "_pag", 0, 0, perm, KEY_ALLOC_NOT_IN_QUOTA); if (!IS_ERR(key)) { - key_instantiate_and_link(key, (void *) newpag, sizeof(afs_uint32), + code = key_instantiate_and_link(key, (void *) newpag, sizeof(afs_uint32), current_session_keyring(), NULL); key_put(key); } else { -- 2.39.5