From owner-cvs-sys Sat May 30 20:15:00 1998 Return-Path: Received: (from daemon@localhost) by hub.freebsd.org (8.8.8/8.8.8) id UAA19737 for cvs-sys-outgoing; Sat, 30 May 1998 20:15:00 -0700 (PDT) (envelope-from owner-cvs-sys) Received: from parkplace.cet.co.jp (parkplace.cet.co.jp [202.32.64.1]) by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id UAA19273; Sat, 30 May 1998 20:11:47 -0700 (PDT) (envelope-from michaelh@cet.co.jp) Received: from localhost (michaelh@localhost) by parkplace.cet.co.jp (8.8.8/CET-v2.2) with SMTP id DAA04648; Sun, 31 May 1998 03:10:44 GMT Date: Sun, 31 May 1998 12:10:44 +0900 (JST) From: Michael Hancock To: Peter Wemm cc: cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG, cvs-sys@FreeBSD.ORG Subject: Re: cvs commit: src/sys/nfs nfs_vnops.c In-Reply-To: <199805310103.SAA07148@freefall.freebsd.org> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-cvs-sys@FreeBSD.ORG X-Loop: FreeBSD.org Precedence: bulk On Sat, 30 May 1998, Peter Wemm wrote: > peter 1998/05/30 18:03:08 PDT > > Modified files: > sys/nfs nfs_vnops.c > Log: > VOP_ABORTUP() appears to be called with the wrong vnode. The other callers > that I checked (eg: ufs_link()) do the ABORTOP on the directory rather than > the file itself. After Michael Hancock's patches, the abortop doesn't seem > all that critial now since something else will free the pathname buffer. These have been done for 2.2 in Terry's nameifree patches. We could use a volunteer to freshen those patches up if Terry doesn't do it himself and abide by KNF.