diff options
| author | Visa Hankala <visa@cvs.openbsd.org> | 2018-04-28 03:13:06 +0000 |
|---|---|---|
| committer | Visa Hankala <visa@cvs.openbsd.org> | 2018-04-28 03:13:06 +0000 |
| commit | 356327a519a213c5d0b435c6d4488b3c5f0b3efc (patch) | |
| tree | 446ed191773422bb8b098be33ad3b13d10a3c43e /sys/kern/kern_sig.c | |
| parent | d8f64152dfaaddf25de9dca020b84c322b874855 (diff) | |
Clean up the parameters of VOP_LOCK() and VOP_UNLOCK(). It is always
curproc that does the locking or unlocking, so the proc parameter
is pointless and can be dropped.
OK mpi@, deraadt@
Diffstat (limited to 'sys/kern/kern_sig.c')
| -rw-r--r-- | sys/kern/kern_sig.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/sys/kern/kern_sig.c b/sys/kern/kern_sig.c index cb448f48280..db7adf619ba 100644 --- a/sys/kern/kern_sig.c +++ b/sys/kern/kern_sig.c @@ -1,4 +1,4 @@ -/* $OpenBSD: kern_sig.c,v 1.219 2018/04/12 17:13:44 deraadt Exp $ */ +/* $OpenBSD: kern_sig.c,v 1.220 2018/04/28 03:13:04 visa Exp $ */ /* $NetBSD: kern_sig.c,v 1.54 1996/04/22 01:38:32 christos Exp $ */ /* @@ -1569,7 +1569,7 @@ coredump(struct proc *p) */ vp = nd.ni_vp; if ((error = VOP_GETATTR(vp, &vattr, cred, p)) != 0) { - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); vn_close(vp, FWRITE, cred, p); goto out; } @@ -1577,7 +1577,7 @@ coredump(struct proc *p) vattr.va_mode & ((VREAD | VWRITE) >> 3 | (VREAD | VWRITE) >> 6) || vattr.va_uid != cred->cr_uid) { error = EACCES; - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); vn_close(vp, FWRITE, cred, p); goto out; } @@ -1590,7 +1590,7 @@ coredump(struct proc *p) io.io_vp = vp; io.io_cred = cred; io.io_offset = 0; - VOP_UNLOCK(vp, p); + VOP_UNLOCK(vp); vref(vp); error = vn_close(vp, FWRITE, cred, p); if (error == 0) |