diff options
| author | kan <kan@FreeBSD.org> | 2009-05-09 18:09:17 +0000 |
|---|---|---|
| committer | kan <kan@FreeBSD.org> | 2009-05-09 18:09:17 +0000 |
| commit | 7b57a857b7ba934dd819c18d33738253d01f87c5 (patch) | |
| tree | e0a6bea48a27765461073d412daf8d760867e537 /sys/nfsserver | |
| parent | 11700d839b55c1904f111a054c18b3ba53a9a98c (diff) | |
| download | FreeBSD-src-7b57a857b7ba934dd819c18d33738253d01f87c5.zip FreeBSD-src-7b57a857b7ba934dd819c18d33738253d01f87c5.tar.gz | |
Do not embed struct ucred into larger netcred parent structures.
Credential might need to hang around longer than its parent and be used
outside of mnt_explock scope controlling netcred lifetime. Use separate
reference-counted ucred allocated separately instead.
While there, extend mnt_explock coverage in vfs_stdexpcheck and clean-up
some unused declarations in new NFS code.
Reported by: John Hickey
PR: kern/133439
Reviewed by: dfr, kib
Diffstat (limited to 'sys/nfsserver')
| -rw-r--r-- | sys/nfsserver/nfs_srvsubs.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/sys/nfsserver/nfs_srvsubs.c b/sys/nfsserver/nfs_srvsubs.c index d154463..3f54664 100644 --- a/sys/nfsserver/nfs_srvsubs.c +++ b/sys/nfsserver/nfs_srvsubs.c @@ -1193,6 +1193,9 @@ nfsrv_fhtovp(fhandle_t *fhp, int lockflag, struct vnode **vpp, int *vfslockedp, if (!lockflag) VOP_UNLOCK(*vpp, 0); out: + if (credanon != NULL) + crfree(credanon); + if (error) { VFS_UNLOCK_GIANT(vfslocked); } else |
