diff options
| author |   Seebs <seebs@seebs.net> | 2017-04-13 18:12:01 -0500 |
|---|---|---|
| committer | Seebs <seebs@seebs.net> | 2017-04-13 18:12:01 -0500 |
| commit | b6b68db896f9963558334aff7fca61adde4ec10f (patch) | |
| tree | 8eb28f2114259cc06af53961b8b60c09d45ac30d /ChangeLog.txt | |
| parent | efe0be279901006f939cd357ccee47b651c786da (diff) | |
| download | pseudo-b6b68db896f9963558334aff7fca61adde4ec10f.tar.gz pseudo-b6b68db896f9963558334aff7fca61adde4ec10f.tar.bz2 pseudo-b6b68db896f9963558334aff7fca61adde4ec10f.zip | |
Prevent bash from segfaulting when unloading pseudo
bash's extremely fancy internal awareness of how the environment looks
means that, if you directly call the underlying libc "unsetenv" on
a variable, bash can end up trying to access a null pointer. Fixing
this generically is actually rather hard; you can't really avoid
writing to environ on fork() or popen(), even if you change all
execv*() functions to use the execv*e() variants. So for now, instead
of unsetting the variable, set it to an empty string.
Thanks to Saur in IRC for spotting this and helping debug it.
Signed-off-by: Seebs <seebs@seebs.net>
Diffstat (limited to 'ChangeLog.txt')
| -rw-r--r-- | ChangeLog.txt | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/ChangeLog.txt b/ChangeLog.txt index a2d30e9..8ba1ffa 100644 --- a/ChangeLog.txt +++ b/ChangeLog.txt @@ -1,3 +1,8 @@ +2017-04-13: + * (seebs) don't unset LD_PRELOAD or the like, because if you + do that, bash can segfault because it "knows" how many + fields are in environ. + 2017-02-24: * (seebs) import posix_acl_default fix from Anton Gerasimov <anton@advancedtelematic.com> |