diff options
author | Hans Mulder <hansm@euronet.nl> | 1997-05-29 20:30:44 +1200 |
---|---|---|
committer | Tim Bunce <Tim.Bunce@ig.co.uk> | 1997-06-11 12:00:00 +1200 |
commit | 66b1d5575fd5eb6242bac2e9a08b163be8b1b960 (patch) | |
tree | 494f32324f1768f55f5912b08ed6164b414f00ae /embed.h | |
parent | 5117ca915f2c72e17e52c313797ad394bd76c418 (diff) | |
download | perl-66b1d5575fd5eb6242bac2e9a08b163be8b1b960.tar.gz |
ENV leaks on win32 (was Re: Comments on ENV patch sought)
Subject: [PATCH] for NETaa13787: %ENV=(); doesn't clear the environment
Perl maintains two representations of the environment:
(A) a hash named %ENV, used by the perl script
(B) a char** named environ, which is passed to child processes
Obviously, the intent is to keep tho two in sync.
This fails in two situations:
(1) A list assignment to %ENV clears (A) but not (B);
(2) Assigning to $0 has the side effect of deleting the key
NoNeSuCh form (B) but not from (A).
$ perl -e '%ENV=(); print "home\n" if exists $ENV{HOME}; exec "echo \$HOME";'
/Users/hansm
$ perl -e '$ENV{NoNeSuCh} = "foo"; $0 = "bar"; exec "echo \$NoNeSuCh";'
$ perl -e '$ENV{NoNeSuCh} = "foo"; exec "echo \$NoNeSuCh";'
foo
$
I've complained about rpoblem (1) before; and Larry assigned
it bug ID NETaa13787 when he entered it into DDTS.
The patch below attempts to remedy both problems, at least on
Unix platforms. I don't know how to handle the environment
on VMS and WIN32; my code simply calls DIE('unimplemented"),
which is honest but won't make users on those plaforms happy.
p5p-msgid: 199705292240.AAA01135@mail.euronet.nl
Signed-off-by: Peter Prymmer <pvhp@forte.com>
Diffstat (limited to 'embed.h')
-rw-r--r-- | embed.h | 1 |
1 files changed, 1 insertions, 0 deletions
@@ -308,6 +308,7 @@ #define lshift_ass_amg Perl_lshift_ass_amg #define lt_amg Perl_lt_amg #define magic_clearenv Perl_magic_clearenv +#define magic_clear_all_env Perl_magic_clear_all_env #define magic_clearpack Perl_magic_clearpack #define magic_clearsig Perl_magic_clearsig #define magic_existspack Perl_magic_existspack |