From owner-cvs-all Fri Oct 6 0: 1:41 2000 Delivered-To: cvs-all@freebsd.org Received: from david.siemens.de (david.siemens.de [192.35.17.14]) by hub.freebsd.org (Postfix) with ESMTP id 2B2AA37B502 for ; Fri, 6 Oct 2000 00:01:38 -0700 (PDT) X-Envelope-Sender-Is: andre.albsmeier@mchp.siemens.de (at relayer david.siemens.de) Received: from mail2.siemens.de (mail2.siemens.de [139.25.208.11]) by david.siemens.de (8.11.0/8.11.0) with ESMTP id e9671WI20113; Fri, 6 Oct 2000 09:01:32 +0200 (MET DST) Received: from curry.mchp.siemens.de (curry.mchp.siemens.de [139.25.42.7]) by mail2.siemens.de (8.11.0/8.11.0) with ESMTP id e9671Vc13972; Fri, 6 Oct 2000 09:01:32 +0200 (MET DST) Received: (from localhost) by curry.mchp.siemens.de (8.11.0/8.11.0) id e9671Vo32193; Date: Fri, 6 Oct 2000 09:01:31 +0200 From: Andre Albsmeier To: Alfred Perlstein Cc: cvs-all@FreeBSD.ORG Subject: Re: cvs commit: src/usr.bin/finger finger.c Message-ID: <20001006090131.A61051@curry.mchp.siemens.de> References: <200010051715.e95HFVn34590@hak.lan.Awfulhak.org> <20001005135833.A87853@citusc17.usc.edu> <20001005142209.G27736@fw.wintelcom.net> <20001006074832.A9078@curry.mchp.siemens.de> <20001005233620.B27736@fw.wintelcom.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5i In-Reply-To: <20001005233620.B27736@fw.wintelcom.net>; from bright@wintelcom.net on Thu, Oct 05, 2000 at 11:36:20PM -0700 X-Echelon: BND CIA NSA Mossad KGB MI6 IRA detonator nuclear assault strike Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG On Thu, 05-Oct-2000 at 23:36:20 -0700, Alfred Perlstein wrote: > * Andre Albsmeier [001005 22:48] wrote: > > On Thu, 05-Oct-2000 at 14:22:09 -0700, Alfred Perlstein wrote: > > > > > > > > You know, perhaps after two security holes we should just > > > > back this darn thing out until someone can review it? > > > > > > I think a good policy about this is to: > > > > > > 1) look at the feature and patch > > > 2) think about it really hard > > > 3) realize how f*cking useless and dangerous it is > > > 4) tell the submitter "no, sorry but i can't possibly put this in!" > > > 5) sit back and have a beer to congratulate yourself after not > > > introducing yet another security issue. > > > > > > http://www.FreeBSD.org/cgi/query-pr.cgi?pr=bin%2F11997 > > > > The patch described in the PR mentioned above is NOT useless. > > Maybe for you since you either got /var/spool/samba and > > /var/spool/lpd each on different 8GB filesystems or because > > all of your 1200 dpi color printers are currently out for > > repair. > > > > Of course, a patch can contain security holes -- especially > > if the submitter mentiones it in the text and is explicitly > > asking for reviewing it. > > > > I wouldn't judge things useless only because _I_ can't use it... > > In the form it was in the PR, ie "huge gaping security hole the > size of Montana" it was entirely useless. > > I should have said "useless/dangerous", not all useless patches > are dangerous but all dangerous patches are useless until corrected. In this case, it would have been enough to write 3) realize how (f*cking) dangerous it is :-). I read your words as if you tried to say that the whole thing is useless but, anyway, this has been cleared out now. -Andre To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message