Environment handling broken in /bin/sh with changes to
{get,set,put}env()
Andrey Chernov
ache at nagual.pp.ru
Sat Jul 7 13:09:03 UTC 2007
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/
More information about the freebsd-current
mailing list