diff options
author | J. Bruce Fields <bfields@fieldses.org> | 2009-01-21 00:34:22 (GMT) |
---|---|---|
committer | J. Bruce Fields <bfields@pig.fieldses.org> | 2009-01-27 22:26:59 (GMT) |
commit | b914152a6fbd2cd0441bc293ae8b3f3f1a9407b6 (patch) | |
tree | 52267a4af3d34ebf190d89b279758d7726383412 | |
parent | bf935a78814cc9b96d09f612912178adc964ce9c (diff) | |
download | linux-fsl-qoriq-b914152a6fbd2cd0441bc293ae8b3f3f1a9407b6.tar.xz |
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>
-rw-r--r-- | fs/nfsd/auth.c | 1 |
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: |