From owner-svn-src-all@FreeBSD.ORG Mon Jan 12 17:54:27 2009 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 3A6641065670; Mon, 12 Jan 2009 17:54:27 +0000 (UTC) (envelope-from obrien@NUXI.org) Received: from dragon.nuxi.org (trang.nuxi.org [74.95.12.85]) by mx1.freebsd.org (Postfix) with ESMTP id 00BAB8FC12; Mon, 12 Jan 2009 17:54:25 +0000 (UTC) (envelope-from obrien@NUXI.org) Received: from dragon.nuxi.org (obrien@localhost [127.0.0.1]) by dragon.nuxi.org (8.14.2/8.14.2) with ESMTP id n0CHsOlY089263; Mon, 12 Jan 2009 09:54:24 -0800 (PST) (envelope-from obrien@dragon.nuxi.org) Received: (from obrien@localhost) by dragon.nuxi.org (8.14.2/8.14.2/Submit) id n0CHsOcx089262; Mon, 12 Jan 2009 09:54:24 -0800 (PST) (envelope-from obrien) Date: Mon, 12 Jan 2009 09:54:24 -0800 From: "David O'Brien" To: Christoph Mallon Message-ID: <20090112175424.GA89144@dragon.NUXI.org> References: <200812262254.mBQMsrbR052676@svn.freebsd.org> <4960FA9A.1090509@gmx.de> <20090111041543.GB17602@dragon.NUXI.org> <4969A626.6070908@gmx.de> <20090112082510.GA69194@dragon.NUXI.org> <496B10F9.20201@gmx.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <496B10F9.20201@gmx.de> X-Operating-System: FreeBSD 8.0-CURRENT User-Agent: Mutt/1.5.16 (2007-06-09) Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r186504 - head/sbin/mount X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: obrien@freebsd.org List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 12 Jan 2009 17:54:27 -0000 On Mon, Jan 12, 2009 at 10:44:25AM +0100, Christoph Mallon wrote: > David O'Brien schrieb: >> On Sun, Jan 11, 2009 at 08:56:22AM +0100, Christoph Mallon wrote: >>> David O'Brien schrieb: >>>> On Sun, Jan 04, 2009 at 07:06:18PM +0100, Christoph Mallon wrote: >>>>> I'm pretty sure $SUPERNATURAL_BEING_OF_YOUR_CHOICE killed a kitten for >>>>> the ugly hack you added to mount. The moment you overflow a buffer, you >>>>> are in no man's land and there's no escape. I appended a patch, which >>>>> solves this issue once and for all: The argv array gets dynamically >>>>> expanded, when its limit is reached. >>>>> Please - for all kittens out there - commit this patch. >>>> Hi Christoph, >>>> Unfortunately your patch doesn't work. >>>> For a 'ufs' file system listed in /etc/fstab >>>> $ umount /foo >>>> $ mount /foo >>>> Does not work. >>> Why haven't you told me earlier? >> Wow, wish I did have the gift to know something before I know something. >> Especially tomorrow's lotto #'s. ;-) > > I sent you my patch almost a week before you commited your changes. I > think, there was enough time to tell me, that my patch had a flaw. Christoph when I work for you for $$, then you can be demanding on when I read email and test the attached patches. Since this is in my free time... > I've attached a corrected version of my patch, which has a mnt_argc = 0; > added in order to reset the argument vector on reentry of mountfs() > (instead of appending to the arguments of the last round). If you want me to read your patches - stop making larger changes than necessary - that primarily undo the changes I've already done. Instead send patches that fix bugs you find in smaller ways (unless that's not possible). thanks, -- -- David (obrien@FreeBSD.org) Q: Because it reverses the logical flow of conversation. A: Why is top-posting (putting a reply at the top of the message) frowned upon? Let's not play "Jeopardy-style quoting"