Re: Environment handling broken in /bin/sh with changes to {get,set,put}env()

[Andrey Chernov <ache_(_at_)_nagual_(_dot_)_pp_(_dot_)_ru>]

On Thu, Jul 05, 2007 at 11:38:53AM -0500, Sean C. Farley wrote:
>> Previously the goal of veryfy_env() is just deactivate, the goal of
>> build_env() is just build. It was build_env() who insetrts new environ
>> variables into envVars array in old variant, isn't?
> 
> Yes, it was.  Now, it is to merge in a new environ array.  I renamed it
> __merge_environ() to better reflect its new role.

Well, I see. You try to keep envVars[] between environ switch by that way. 

But it still look complicated and probably gains nothing. I.e. will be 
much _faster_ just free envVars[] (but not variables themselfs) and allow 
build_env() to calloc() new array for envVars and fills it from new 
environ. It is surely faster than calling setenv() for each variable just 
for sake of keepeng once allocated envVars[].

Moreover, environ switch commonly used to switch from large environ to 
smaller one (or to empty one), so the rest of old envVars[] array would 
keep unneccessary allocation.

> The alternative, which I had actually considered, is to split setenv()
> into __setenv() which is almost the entire current setenv() and a new
> setenv() that is just a wrapper around __setenv() with the beginning
> checks.  This seems a bit of a waste, but I may be mistaken.

Well, no recurse level increase problems but just slowdown.

-- 
http://ache.pp.ru/
_______________________________________________
freebsd-current_(_at_)_freebsd_(_dot_)_org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscribe_(_at_)_freebsd_(_dot_)_org"