diff options
author | rwatson <rwatson@FreeBSD.org> | 2002-02-10 21:44:30 +0000 |
---|---|---|
committer | rwatson <rwatson@FreeBSD.org> | 2002-02-10 21:44:30 +0000 |
commit | 2bbae54d18d15c9c93f9567d79fe63538dd04bfd (patch) | |
tree | 069df193b86466f9270600030958b981aceda286 /sys/kern/vfs_vnops.c | |
parent | ef612b7ce49f905c66a8d08360c010cbcbd07aba (diff) | |
download | FreeBSD-src-2bbae54d18d15c9c93f9567d79fe63538dd04bfd.zip FreeBSD-src-2bbae54d18d15c9c93f9567d79fe63538dd04bfd.tar.gz |
Make sure to hold vnode lock when calling into VOP_GETATTR().
Discussed with: mckusick, phk
Diffstat (limited to 'sys/kern/vfs_vnops.c')
-rw-r--r-- | sys/kern/vfs_vnops.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/sys/kern/vfs_vnops.c b/sys/kern/vfs_vnops.c index d7c4bc8..e9b3e90 100644 --- a/sys/kern/vfs_vnops.c +++ b/sys/kern/vfs_vnops.c @@ -485,8 +485,13 @@ vn_statfile(fp, sb, td) struct thread *td; { struct vnode *vp = (struct vnode *)fp->f_data; + int error; - return vn_stat(vp, sb, td); + vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, td); + error = vn_stat(vp, sb, td); + VOP_UNLOCK(vp, 0, td); + + return (error); } int @@ -617,7 +622,9 @@ vn_ioctl(fp, com, data, td) case VREG: case VDIR: if (com == FIONREAD) { + vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, td); error = VOP_GETATTR(vp, &vattr, td->td_proc->p_ucred, td); + VOP_UNLOCK(vp, 0, td); if (error) return (error); *(int *)data = vattr.va_size - fp->f_offset; |