diff options
author | tjr <tjr@FreeBSD.org> | 2002-12-15 04:32:41 +0000 |
---|---|---|
committer | tjr <tjr@FreeBSD.org> | 2002-12-15 04:32:41 +0000 |
commit | fdf35b6cdb02da94956ac62c984a745d410c4db2 (patch) | |
tree | 4c7e63afb7ddaefb7529d6fa0ae0f39f0528088f /sys/fs/smbfs/smbfs_node.c | |
parent | 1a902e59d0765c068a2911ef9429d396a32b982a (diff) | |
download | FreeBSD-src-fdf35b6cdb02da94956ac62c984a745d410c4db2.zip FreeBSD-src-fdf35b6cdb02da94956ac62c984a745d410c4db2.tar.gz |
Remove redundant check for negative or zero v_usecount; vrele() already
checks that.
Diffstat (limited to 'sys/fs/smbfs/smbfs_node.c')
-rw-r--r-- | sys/fs/smbfs/smbfs_node.c | 21 |
1 files changed, 7 insertions, 14 deletions
diff --git a/sys/fs/smbfs/smbfs_node.c b/sys/fs/smbfs/smbfs_node.c index 0feadd0..093e7ca4 100644 --- a/sys/fs/smbfs/smbfs_node.c +++ b/sys/fs/smbfs/smbfs_node.c @@ -317,20 +317,13 @@ smbfs_reclaim(ap) if (np->n_name) smbfs_name_free(np->n_name); FREE(np, M_SMBNODE); - if (dvp) { - VI_LOCK(dvp); - if (dvp->v_usecount >= 1) { - VI_UNLOCK(dvp); - vrele(dvp); - /* - * Indicate that we released something; see comment - * in smbfs_unmount(). - */ - smp->sm_didrele = 1; - } else { - VI_UNLOCK(dvp); - SMBERROR("BUG: negative use count for parent!\n"); - } + if (dvp != NULL) { + vrele(dvp); + /* + * Indicate that we released something; see comment + * in smbfs_unmount(). + */ + smp->sm_didrele = 1; } return 0; } |