aboutsummaryrefslogtreecommitdiff
path: root/selinux/runcon.c
diff options
context:
space:
mode:
authorGravatar Denys Vlasenko <vda.linux@googlemail.com>2016-08-19 20:22:45 +0200
committerGravatar Denys Vlasenko <vda.linux@googlemail.com>2016-08-19 20:24:49 +0200
commitca26f1c238c51b3b2f80dd25fd7fcf8774f38ca4 (patch)
tree77765ce82ef68797ff53fb0efede7ff45bd2b932 /selinux/runcon.c
parent8191abd146989652ce376f0e36c5e1adbea99fe1 (diff)
downloadbusybox-ca26f1c238c51b3b2f80dd25fd7fcf8774f38ca4.tar.gz
busybox-ca26f1c238c51b3b2f80dd25fd7fcf8774f38ca4.tar.bz2
hush: fix a bug with "stdio rewind on exit". Closes 9186
With FEATURE_SH_STANDALONE=y, run this in a "sh SCRIPT": sha256sum /dev/null echo END sha256sum is a NOEXEC applet. It runs in a forked child. Then child exit()s. By this time, entire script is read, and buffered in a FILE object from fopen("SCRIPT"). But fgetc() did not consume entire input. exit() lseeks back by -9 bytes, from <eof> to 'e' in 'echo'. (this may be libc-specific). This change of fd position *is shared with the parent*! Now parent can read more, and it thinks there is another "echo END". End result: two "echo END"s are run. `cmd` and arithmetic also need the fix for this, even without FEATURE_SH_STANDALONE. Fix this by _exit()ing instead. Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'selinux/runcon.c')
0 files changed, 0 insertions, 0 deletions