summaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorJ. Bruce Fields <bfields@fieldses.org>2009-01-20 19:34:22 -0500
committerJ. Bruce Fields <bfields@pig.fieldses.org>2009-01-27 17:26:59 -0500
commitb914152a6fbd2cd0441bc293ae8b3f3f1a9407b6 (patch)
tree52267a4af3d34ebf190d89b279758d7726383412 /fs
parentbf935a78814cc9b96d09f612912178adc964ce9c (diff)
downloadop-kernel-dev-b914152a6fbd2cd0441bc293ae8b3f3f1a9407b6.zip
op-kernel-dev-b914152a6fbd2cd0441bc293ae8b3f3f1a9407b6.tar.gz
nfsd: fix cred leak on every rpc
Since override_creds() took its own reference on new, we need to release our own reference. (Note the put_cred on the return value puts the *old* value of current->creds, not the new passed-in value). Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Diffstat (limited to 'fs')
-rw-r--r--fs/nfsd/auth.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/nfsd/auth.c b/fs/nfsd/auth.c
index b860d34..5573508 100644
--- a/fs/nfsd/auth.c
+++ b/fs/nfsd/auth.c
@@ -87,6 +87,7 @@ int nfsd_setuser(struct svc_rqst *rqstp, struct svc_export *exp)
new->cap_effective = cap_raise_nfsd_set(new->cap_effective,
new->cap_permitted);
put_cred(override_creds(new));
+ put_cred(new);
return 0;
oom:
OpenPOWER on IntegriCloud