From suiram@port294.megapop.eunet.no  Fri Oct  6 21:34:20 2000
Return-Path: <suiram@port294.megapop.eunet.no>
Received: from port294.megapop.eunet.no (port294.megapop.eunet.no [193.91.253.44])
	by hub.freebsd.org (Postfix) with ESMTP id E41F137B503
	for <FreeBSD-gnats-submit@freebsd.org>; Fri,  6 Oct 2000 21:34:17 -0700 (PDT)
Received: (from suiram@localhost)
	by suiram.freebsd.org (8.9.3/8.9.3) id EAA23533;
	Sat, 7 Oct 2000 04:07:23 +0200 (CEST)
	(envelope-from suiram)
Message-Id: <200010070207.EAA23533@suiram.freebsd.org>
Date: Sat, 7 Oct 2000 04:07:23 +0200 (CEST)
From: mbendiks@eunet.no
Sender: suiram@port294.megapop.eunet.no
Reply-To: mbendiks@eunet.no
To: FreeBSD-gnats-submit@freebsd.org
Subject: [patches] Fix erronous comment in msdosfs
X-Send-Pr-Version: 3.2

>Number:         21805
>Category:       kern
>Synopsis:       msdosfs_vnops comments are incorrect wrt locking
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    freebsd-bugs
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          doc-bug
>Submitter-Id:   current-users
>Arrival-Date:   Fri Oct 06 21:40:01 PDT 2000
>Closed-Date:    Sun Oct 22 07:24:33 PDT 2000
>Last-Modified:  Sun Oct 22 07:25:31 PDT 2000
>Originator:     Marius Bendiksen
>Release:        FreeBSD 4.1-RELEASE i386
>Organization:
n/a
>Environment:

	not relevant.

>Description:

	In /sys/msdosfs/msdosfs_vnops.c, function msdosfs_rename(), the
	comments erronously claim that the parent is locked. This isn't
	correct, according to the UFS code and documentation. Also, the
	code doesn't do it anyway, so someone probably just forgot it.

>How-To-Repeat:

	not relevant.

>Fix:

	diff -u4 in /sys/msdosfs, output follows:

--- msdosfs_vnops.c.orig	Sun Sep 24 14:20:20 2000
+++ msdosfs_vnops.c	Sun Sep 24 14:21:54 2000
@@ -865,11 +865,9 @@
 	return (error);
 }
 
 /*
- * DOS filesystems don't know what links are. But since we already called
- * msdosfs_lookup() with create and lockparent, the parent is locked so we
- * have to free it before we return the error.
+ * DOS filesystems don't know what links are.
  */
 static int
 msdosfs_link(ap)
 	struct vop_link_args /* {

>Release-Note:
>Audit-Trail:
State-Changed-From-To: open->closed 
State-Changed-By: bp 
State-Changed-When: Sun Oct 22 07:24:33 PDT 2000 
State-Changed-Why:  
Committed, thanks! 

http://www.freebsd.org/cgi/query-pr.cgi?pr=21805 
>Unformatted:
