diff options
author | Denys Vlasenko | 2017-07-06 20:57:37 +0200 |
---|---|---|
committer | Denys Vlasenko | 2017-07-06 20:57:37 +0200 |
commit | 50b8b2914b7551b4e36518fcc70aac201d46d7cb (patch) | |
tree | 0c8c9a65c10fec5323d0ff8dd0374b89c5d12d0f /shell/hush.c | |
parent | a107ef2a6ace98c51473dc3153564a44b260bc6f (diff) | |
download | busybox-50b8b2914b7551b4e36518fcc70aac201d46d7cb.zip busybox-50b8b2914b7551b4e36518fcc70aac201d46d7cb.tar.gz |
hush: add a TODO about redir3.tests failure
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'shell/hush.c')
-rw-r--r-- | shell/hush.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/shell/hush.c b/shell/hush.c index 0ade2cc..4ba6b3f 100644 --- a/shell/hush.c +++ b/shell/hush.c @@ -7723,6 +7723,10 @@ static NOINLINE int run_pipe(struct pipe *pi) unset_vars(new_env); add_vars(old_vars); /* clean_up_and_ret0: */ + +//FIXME: this restores stdio fds, but does not close other redirects! +//Example: after "echo TEST 9>/dev/null" fd#9 is not closed! +//The squirreling code needs rework to remember all fds, not just 0,1,2. restore_redirects(squirrel); clean_up_and_ret1: free(argv_expanded); |