diff options
| author | kib <kib@FreeBSD.org> | 2011-10-24 13:53:32 +0000 |
|---|---|---|
| committer | kib <kib@FreeBSD.org> | 2011-10-24 13:53:32 +0000 |
| commit | 177754802c82adf2c6928cda80086b32ccd1e8eb (patch) | |
| tree | 4039c627a41c136001f1a2c091ef2bdfc485ca6a /sys/fs/nullfs/null_vfsops.c | |
| parent | 7950dabcc02032cfcac6897c547a4b5321995a2a (diff) | |
| download | FreeBSD-src-177754802c82adf2c6928cda80086b32ccd1e8eb.zip FreeBSD-src-177754802c82adf2c6928cda80086b32ccd1e8eb.tar.gz | |
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
Diffstat (limited to 'sys/fs/nullfs/null_vfsops.c')
| -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 7eaffe5..fd20151 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); |
