diff options
author | pjd <pjd@FreeBSD.org> | 2007-07-26 16:58:09 +0000 |
---|---|---|
committer | pjd <pjd@FreeBSD.org> | 2007-07-26 16:58:09 +0000 |
commit | fe74e944d12452785490f4343dd388287bc64da4 (patch) | |
tree | 5a98f364f3abdf11cc44ba4d0cfc643ff591bd63 /sys/kern/uipc_usrreq.c | |
parent | 8dc8d42bdcbcf383b19e382acdec9adc0bd70d90 (diff) | |
download | FreeBSD-src-fe74e944d12452785490f4343dd388287bc64da4.zip FreeBSD-src-fe74e944d12452785490f4343dd388287bc64da4.tar.gz |
When we do open, we should lock the vnode exclusively. This fixes few races:
- fifo race, where two threads assign v_fifoinfo,
- v_writecount modifications,
- v_object modifications,
- and probably more...
Discussed with: kib, ups
Approved by: re (rwatson)
Diffstat (limited to 'sys/kern/uipc_usrreq.c')
-rw-r--r-- | sys/kern/uipc_usrreq.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/kern/uipc_usrreq.c b/sys/kern/uipc_usrreq.c index 9acb596..682496e 100644 --- a/sys/kern/uipc_usrreq.c +++ b/sys/kern/uipc_usrreq.c @@ -463,7 +463,7 @@ restart: goto error; } vp = nd.ni_vp; - ASSERT_VOP_LOCKED(vp, "uipc_bind"); + ASSERT_VOP_ELOCKED(vp, "uipc_bind"); soun = (struct sockaddr_un *)sodupsockaddr(nam, M_WAITOK); UNP_GLOBAL_WLOCK(); |