diff options
| author |   oga <oga@openbsd.org> | 2010-09-23 18:43:37 +0000 |
|---|---|---|
| committer | oga <oga@openbsd.org> | 2010-09-23 18:43:37 +0000 |
| commit | 10af85c94bf39b02f173c904d037c6e4f911bdef (patch) | |
| tree | 31f878ab29a782a1403a729ce6eec65043711b03 | |
| parent | Change: (diff) | |
| download | wireguard-openbsd-10af85c94bf39b02f173c904d037c6e4f911bdef.tar.xz wireguard-openbsd-10af85c94bf39b02f173c904d037c6e4f911bdef.zip | |
In procfs_root(), while the vn_lock on a newly-allocated vnode can not
fail it is still good practise to either check the return value or to
provide a flag that won't (to avoid copy pasted code that gets it
wrong).
So change LK_EXCLUSIVE to LK_EXCLUSIVE | LK_RETRY, removing the one
caller of vn_lock in the tree that doesn't either pass LK_RETRY or check
the return value.
ok thib@, jsing@ as part of a larger diff.
| -rw-r--r-- | sys/miscfs/procfs/procfs_vfsops.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/miscfs/procfs/procfs_vfsops.c b/sys/miscfs/procfs/procfs_vfsops.c index 637786a675e..2325b8a4e17 100644 --- a/sys/miscfs/procfs/procfs_vfsops.c +++ b/sys/miscfs/procfs/procfs_vfsops.c @@ -1,4 +1,4 @@ -/* $OpenBSD: procfs_vfsops.c,v 1.25 2007/06/18 08:30:07 jasper Exp $ */ +/* $OpenBSD: procfs_vfsops.c,v 1.26 2010/09/23 18:43:37 oga Exp $ */ /* $NetBSD: procfs_vfsops.c,v 1.25 1996/02/09 22:40:53 christos Exp $ */ /* @@ -149,7 +149,7 @@ procfs_root(struct mount *mp, struct vnode **vpp) error = procfs_allocvp(mp, vpp, 0, Proot); if (error) return (error); - vn_lock(*vpp, LK_EXCLUSIVE, curproc); + vn_lock(*vpp, LK_EXCLUSIVE | LK_RETRY, curproc); return (0); } |