diff options
author | kib <kib@FreeBSD.org> | 2006-09-19 14:04:12 +0000 |
---|---|---|
committer | kib <kib@FreeBSD.org> | 2006-09-19 14:04:12 +0000 |
commit | cf9722d790626e547970c71bd007d48a02bfd70e (patch) | |
tree | 8c511f2a5ef0e19f841e024c7e80bde8f972ffc9 | |
parent | 11200e2de3b5360427a73ee6ff445ce62a785119 (diff) | |
download | FreeBSD-src-cf9722d790626e547970c71bd007d48a02bfd70e.zip FreeBSD-src-cf9722d790626e547970c71bd007d48a02bfd70e.tar.gz |
Fix the bug in rev. 1.232. If vfs_suser returned false, coveredvp shall be
unlocked only if it really exists.
Found with: Coverity Prevent(tm)
CID: 1535
Approved by: pjd (mentor)
-rw-r--r-- | sys/kern/vfs_mount.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sys/kern/vfs_mount.c b/sys/kern/vfs_mount.c index 486c8a1..64c64c9 100644 --- a/sys/kern/vfs_mount.c +++ b/sys/kern/vfs_mount.c @@ -1151,7 +1151,8 @@ dounmount(mp, flags, td) */ error = vfs_suser(mp, td); if (error) { - VOP_UNLOCK(coveredvp, 0, td); + if (coveredvp) + VOP_UNLOCK(coveredvp, 0, td); return (error); } |