vfs: whack dpunlocked var in vfs_lookup

It is redundant given the bad_unlocked goto label.
This commit is contained in:
Mateusz Guzik 2023-07-05 21:52:07 +00:00
parent 544deacc90
commit 0724cf3862

View file

@ -926,7 +926,6 @@ vfs_lookup(struct nameidata *ndp)
int wantparent; /* 1 => wantparent or lockparent flag */
int rdonly; /* lookup read-only flag bit */
int error = 0;
int dpunlocked = 0; /* dp has already been unlocked */
int relookup = 0; /* do not consume the path component */
struct componentname *cnp = &ndp->ni_cnd;
int lkflags_save;
@ -1342,10 +1341,8 @@ good:
vput(dp);
if (vn_lock(vp_crossmp, LK_SHARED | LK_NOWAIT))
panic("vp_crossmp exclusively locked or reclaimed");
if (error != 0) {
dpunlocked = 1;
goto bad2;
}
if (error != 0)
goto bad_unlocked;
ndp->ni_vp = dp = tdp;
} while ((vn_irflag_read(dp) & VIRF_MOUNTPOINT) != 0);
@ -1457,8 +1454,7 @@ bad2:
vrele(ndp->ni_dvp);
}
bad:
if (!dpunlocked)
vput(dp);
vput(dp);
bad_unlocked:
ndp->ni_vp = NULL;
return (error);