diff options
author | Konstantin Belousov <kib@FreeBSD.org> | 2011-10-24 13:53:32 +0000 |
---|---|---|
committer | Konstantin Belousov <kib@FreeBSD.org> | 2011-10-24 13:53:32 +0000 |
commit | 234ab7412e398025a9669055abb1085644347275 (patch) | |
tree | 4039c627a41c136001f1a2c091ef2bdfc485ca6a /sys | |
parent | ffa43617e8486ba6e7ba8542958b4036650303b7 (diff) | |
download | src-234ab7412e398025a9669055abb1085644347275.tar.gz src-234ab7412e398025a9669055abb1085644347275.zip |
The only possible error return from null_nodeget() is due to insmntque1
failure (the getnewvnode cannot return an error). In this case, the
null_insmntque_dtr() already unlocked the reclaimed vnode, so VOP_UNLOCK()
in the nullfs_mount() after null_nodeget() failure is wrong.
Tested by: pho
MFC after: 1 week
Notes
Notes:
svn path=/head/; revision=226687
Diffstat (limited to 'sys')
-rw-r--r-- | sys/fs/nullfs/null_vfsops.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/sys/fs/nullfs/null_vfsops.c b/sys/fs/nullfs/null_vfsops.c index 7eaffe56dd3d..fd20151f7c55 100644 --- a/sys/fs/nullfs/null_vfsops.c +++ b/sys/fs/nullfs/null_vfsops.c @@ -157,7 +157,6 @@ nullfs_mount(struct mount *mp) * Make sure the node alias worked */ if (error) { - VOP_UNLOCK(vp, 0); vrele(lowerrootvp); free(xmp, M_NULLFSMNT); /* XXX */ return (error); |