NetBSD Problem Report #43745

From www@NetBSD.org  Thu Aug 12 09:37:40 2010
Return-Path: <www@NetBSD.org>
Received: from mail.netbsd.org (mail.netbsd.org [204.152.190.11])
	by www.NetBSD.org (Postfix) with ESMTP id AD95663BBBD
	for <gnats-bugs@gnats.NetBSD.org>; Thu, 12 Aug 2010 09:37:40 +0000 (UTC)
Message-Id: <20100812093740.74E9763BBBA@www.NetBSD.org>
Date: Thu, 12 Aug 2010 09:37:40 +0000 (UTC)
From: pooka@iki.fi
Reply-To: pooka@iki.fi
To: gnats-bugs@NetBSD.org
Subject: fhopen of an unlinked file causes problems on multiple file systems
X-Send-Pr-Version: www-1.0

>Number:         43745
>Category:       kern
>Synopsis:       fhopen of an unlinked file causes problems on multiple file systems
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    kern-bug-people
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Thu Aug 12 09:40:00 +0000 2010
>Last-Modified:  Mon Apr 04 19:20:02 +0000 2011
>Originator:     Antti Kantee
>Release:        
>Organization:
>Environment:
>Description:
getfh() + unlink() + fhopen() succeeds on msdosfs and lfs.
It panics the kernel on with nfs.
>How-To-Repeat:
run tests/fs/vfs/t_vfsops
>Fix:

>Audit-Trail:
From: Antti Kantee <pooka@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/43745 CVS commit: src/sys/nfs
Date: Thu, 12 Aug 2010 09:41:11 +0000

 Module Name:	src
 Committed By:	pooka
 Date:		Thu Aug 12 09:41:11 UTC 2010

 Modified Files:
 	src/sys/nfs: nfs_vfsops.c

 Log Message:
 Do not return a garbage vnode in vpp if fhtovp fails.

 Fixes PR kern/43745 for nfs.


 To generate a diff of this commit:
 cvs rdiff -u -r1.216 -r1.217 src/sys/nfs/nfs_vfsops.c

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

From: Antti Kantee <pooka@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/43745 CVS commit: src/tests/fs/vfs
Date: Thu, 12 Aug 2010 09:42:53 +0000

 Module Name:	src
 Committed By:	pooka
 Date:		Thu Aug 12 09:42:53 UTC 2010

 Modified Files:
 	src/tests/fs/vfs: t_vfsops.c

 Log Message:
 Fill in PR kern/43745 now that it exists.

     lfs_tfhremove: Expected failure: fhopen() for removed file succeeds (PR kern/43745): t_vfsops.c:161: Expected true value in rump_sys_fhopen(fhp, fhsize, O_RDONLY) == -1
     msdosfs_tfhremove: Expected failure: fhopen() for removed file succeeds (PR kern/43745): t_vfsops.c:161: Expected true value in rump_sys_fhopen(fhp, fhsize, O_RDONLY) == -1


 To generate a diff of this commit:
 cvs rdiff -u -r1.5 -r1.6 src/tests/fs/vfs/t_vfsops.c

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

From: "Juergen Hannken-Illjes" <hannken@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/43745 CVS commit: src
Date: Mon, 4 Apr 2011 19:16:59 +0000

 Module Name:	src
 Committed By:	hannken
 Date:		Mon Apr  4 19:16:58 UTC 2011

 Modified Files:
 	src/sys/fs/msdosfs: denode.h msdosfs_denode.c msdosfs_vfsops.c
 	src/tests/fs/vfs: t_vfsops.c

 Log Message:
 Msdosfs on-disk meta data is not sufficient to create or validate file handles.

 Maintain a tree of file handles, create nodes from msdosfs_vptofh() and keep
 them until either the file gets unlinked or the file system gets unmounted.

 Fixes the msdosfs part of PR #43745 (fhopen of an unlinked file causes problems
 on multiple file systems)


 To generate a diff of this commit:
 cvs rdiff -u -r1.18 -r1.19 src/sys/fs/msdosfs/denode.h
 cvs rdiff -u -r1.42 -r1.43 src/sys/fs/msdosfs/msdosfs_denode.c
 cvs rdiff -u -r1.89 -r1.90 src/sys/fs/msdosfs/msdosfs_vfsops.c
 cvs rdiff -u -r1.10 -r1.11 src/tests/fs/vfs/t_vfsops.c

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

Home	PR Database Search