PR/41147 CVS commit: src/sys/ufs/ufs

[Manuel Bouyer <bouyer%netbsd.org@localhost>]

The following reply was made to PR kern/41147; it has been noted by GNATS.

From: Manuel Bouyer <bouyer%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/41147 CVS commit: src/sys/ufs/ufs
Date: Sun, 20 Sep 2009 14:00:24 +0000

 Module Name:   src
 Committed By:  bouyer
 Date:          Sun Sep 20 14:00:24 UTC 2009
 
 Modified Files:
        src/sys/ufs/ufs: ufs_ihash.c
 
 Log Message:
 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 -r1.27 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.