Date: Fri, 11 May 2007 04:34:44 +0400 From: Andrey Chernov <ache@FreeBSD.org> To: "Sean C. Farley" <sean-freebsd@farley.org> Cc: Daniel Eischen <deischen@FreeBSD.org>, arch@FreeBSD.org Subject: Re: HEADS DOWN Message-ID: <20070511003443.GA6422@nagual.pp.ru> In-Reply-To: <20070510184447.H4969@baba.farley.org> References: <20070504213312.GA33163@nagual.pp.ru> <20070504174657.D1343@thor.farley.org> <20070505213202.GA49925@nagual.pp.ru> <20070505163707.J6670@thor.farley.org> <20070505221125.GA50439@nagual.pp.ru> <20070506091835.A43775@besplex.bde.org> <20070508162458.G6015@baba.farley.org> <20070508222521.GA59534@nagual.pp.ru> <20070509200000.B56490@besplex.bde.org> <20070510184447.H4969@baba.farley.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, May 10, 2007 at 06:58:45PM -0500, Sean C. Farley wrote: > Would there be any other changes anybody can see need to be made? What > type of testing would be desired? The regression tests I wrote provide > a good basic test. I worry about this sort of things errx(EXIT_FAILURE, "environ corrupt"); There is no mention anywhere that *env() functions can exit the program. Moreover some programs in theory can temprorarily put incorrect values into environment via putenv() after-modification or direct environ assignments for their own reasons. I suggest to change errx() to warnx()+return(failure). -- http://ache.pp.ru/
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20070511003443.GA6422>