summaryrefslogtreecommitdiff
path: root/archival/bz/bzlib.c
diff options
context:
space:
mode:
authorColin Watson2010-04-26 14:21:27 +0200
committerDenys Vlasenko2010-04-26 14:21:27 +0200
commit3963d943f8a9b4ef89f000b5e1424e63dba920f5 (patch)
treeaddf83c843ab6b0651c08bee282379541bc8fd9b /archival/bz/bzlib.c
parent920185c69ac9addf06cb47a0f2336e075d6dacb2 (diff)
downloadbusybox-3963d943f8a9b4ef89f000b5e1424e63dba920f5.zip
busybox-3963d943f8a9b4ef89f000b5e1424e63dba920f5.tar.gz
ash: refresh stack pointers after makestrspace in rmescapes
Without this, rmescapes sometimes returns random garbage while performing parameter expansions such as ${foo#bar}, in the event that the allocation of r happens to need to reallocate the stack and hence invalidate str and p. I'd love to provide a test case but unfortunately it's dependent on exact stack layout, so I don't have anything simpler than the situation described in https://bugs.launchpad.net/ubuntu/+source/partman-base/+bug/527401/comments/23 which involved a sequence of foo="${foo#*, }" expansions on a long string inside our RAID configuration tool. The same fix has been in dash since 2007-09-26, contributed by Roy Marples <uberlord@gentoo.org>. I actually came up with it independently almost to the character, but then synced it up with the variable naming used in dash when I noticed that change afterwards. Signed-off-by: Colin Watson <cjwatson@ubuntu.com> Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'archival/bz/bzlib.c')
0 files changed, 0 insertions, 0 deletions