From owner-cvs-all Fri Jun 2 8:34:47 2000 Delivered-To: cvs-all@freebsd.org Received: from khavrinen.lcs.mit.edu (khavrinen.lcs.mit.edu [18.24.4.193]) by hub.freebsd.org (Postfix) with ESMTP id 00B2B37BDEB; Fri, 2 Jun 2000 08:34:38 -0700 (PDT) (envelope-from wollman@khavrinen.lcs.mit.edu) Received: (from wollman@localhost) by khavrinen.lcs.mit.edu (8.9.3/8.9.3) id LAA41585; Fri, 2 Jun 2000 11:34:37 -0400 (EDT) (envelope-from wollman) Date: Fri, 2 Jun 2000 11:34:37 -0400 (EDT) From: Garrett Wollman Message-Id: <200006021534.LAA41585@khavrinen.lcs.mit.edu> To: Chris Costello Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/miscfs/fdesc fdesc.h fdesc_vnops.c In-Reply-To: <200006020702.AAA11016@freefall.freebsd.org> References: <200006020702.AAA11016@freefall.freebsd.org> Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG < said: > - Do not allow VOP_SETATTR to modify underlying vnodes at all. This caused > problems when fetch(1) was passed `-o -'. The rationale of this change > is that applications attempting to change underlying vnodes for /dev/fd > nodes are improperly written and the use of this interface should not > ever have been encouraged. Proper alternatives are fchmod, fchown and > others. Huh? The only operations `fetch' *ever* performs on /dev/stdout are: fopen fseek fwrite fread (iff the server provides an MD5) (which will fail anyway) I'd make the additional argument that this change is bogus because: If fetch had been passed `-o /dev/fd/69', it would be unreasonable to expect fetch to examine the argument it was given, notice that it began with the magic string `/dev/fd/' and parse out the file-descriptor number. This is not to say that there are no bugs in `fetch' -- I found a couple in the vicinity of rm() just by inspection, one of which actually bit me once. (Hint: never interrupt fetch while running as root with `-o /dev/null' -- it will delete the device node.) -GAWollman -- Garrett A. Wollman | O Siem / We are all family / O Siem / We're all the same wollman@lcs.mit.edu | O Siem / The fires of freedom Opinions not those of| Dance in the burning flame MIT, LCS, CRS, or NSA| - Susan Aglukark and Chad Irschick To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message