cvs commit: src/usr.sbin/sysinstall main.c

John Baldwin jhb at freebsd.org
Mon Apr 30 17:37:53 UTC 2007


On Monday 30 April 2007 12:40:31 pm Andrey Chernov wrote:
> On Mon, Apr 30, 2007 at 12:29:20PM -0400, John Baldwin wrote:
> > Have you coordinated at all with the guy on current@ who has patches to 
make 
> > setenv(3) not leak memory as bad?  
> 
> No, I don't touch current allocation scheme at all. It isn't my goal.
> 
> > Also, given that we malloc a limited space 
> > for the string values, I don't see how you can make it so that one can 
always 
> > just overwrite the string pointed to by putenv(3)'s return value to change 
> > the value.  If we malloc a buffer for length N and the user wants to set 
the 
> > length to M > N, we pretty much have to malloc a new buffer that will end 
up 
> > at a different address, so places holding onto the previous value returned 
> > from putenv(3) will stop seeing updates.
> 
> It isn't the issue. Putenv value supposed to live just up to the next 
> putenv or setenv call, so setenv can legitimately overwrite it.

Ok.  FWIW, this seems like a ridiculous and gross hack just to provide a 
backdoor for updating the environment w/o making a fooenv() function call 
(either putenv, or setenv).

-- 
John Baldwin


More information about the cvs-src mailing list