Linux: crset should use passed credentials
authorMarc Dionne <marc.c.dionne@gmail.com>
Fri, 25 Jan 2013 02:38:28 +0000 (21:38 -0500)
committerDerrick Brashear <shadow@your-file-system.com>
Tue, 5 Feb 2013 12:21:29 +0000 (04:21 -0800)
When copying credentials in crset, take the passed credentials as
the source instead of the current process credentials.

This doesn't cause any problems currently since crset always gets
called with the current credentials.

Change-Id: Ia6a341e96d00071587443f8eef4a99992b368abc
Reviewed-on: http://gerrit.openafs.org/8980
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>

src/afs/LINUX/osi_cred.c

index 20be594..f271321 100644 (file)
@@ -117,7 +117,7 @@ crset(cred_t * cr)
     new_creds = prepare_creds();
     /* Drop the reference to group_info - we'll overwrite it in afs_copy_creds */
     put_group_info(new_creds->group_info);
-    afs_copy_creds(new_creds, current_cred());
+    afs_copy_creds(new_creds, cr);
 
     commit_creds(new_creds);
 #else