[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
PR/41147 CVS commit: [netbsd-5] src/sys/ufs/ufs
The following reply was made to PR kern/41147; it has been noted by GNATS.
From: Soren Jacobsen <snj%netbsd.org@localhost>
Subject: PR/41147 CVS commit: [netbsd-5] src/sys/ufs/ufs
Date: Mon, 28 Sep 2009 01:43:02 +0000
Module Name: src
Committed By: snj
Date: Mon Sep 28 01:43:02 UTC 2009
src/sys/ufs/ufs [netbsd-5]: ufs_ihash.c
Pull up following revision(s) (requested by bouyer in ticket #1029):
sys/ufs/ufs/ufs_ihash.c: revision 1.27
PR kern/41147: race between nfsd and local rm
Note that the race also exists between 2 nfs client, one of them doing the rm.
In ufs_ihashget(), vget() can return a vnode that has been vclean'ed because
vget() can sleep. After vget returns, check that vp is still connected with
ip, and that ip still points to the inode we want. This fix the NULL
pointer dereference in ufs_fhtovp() I've been seeing on a NFS server.
XXX I have no idea why using vput() instead of
vlockmgr(vp->v_vnlock, LK_RELEASE); vrele(vp); does not work.
To generate a diff of this commit:
cvs rdiff -u -r1.26 -r18.104.22.168 src/sys/ufs/ufs/ufs_ihash.c
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
Main Index |
Thread Index |