projects / oweals / busybox.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: fe93624) | patch
ash: fix open fds leaking in redirects. Closes 9561
authorDenys Vlasenko <vda.linux@googlemail.com>
Sat, 7 Jan 2017 09:15:01 +0000 (10:15 +0100)
committerDenys Vlasenko <vda.linux@googlemail.com>
Sat, 7 Jan 2017 09:16:56 +0000 (10:16 +0100)
commit86584e134eec1a81298149f8c04c77727f6dccb9
tree6e3323d82c8dc1c2c7da58fa86573ca1cea33e71tree | snapshot
parentfe93624fb69e3858ebaea2d9e53e0522c1025698commit | diff
ash: fix open fds leaking in redirects. Closes 9561

commit e19923f6652a638ac39c84012e97f52cf5a7568e deleted clearredir()
call in shellexec():

ash: [REDIR] Remove redundant CLOEXEC calls
Upstream commit:

Now that we're marking file descriptors as CLOEXEC in savefd, we no longer
need to close them on exec or in setinputfd.

but it missed one place where we don't set CLOEXEC. Fixing this.

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
shell/ash.c diff | blob | history
shell/ash_test/ash-redir/redir_leak.right [new file with mode: 0644] blob
shell/ash_test/ash-redir/redir_leak.tests [new file with mode: 0755] blob
shell/hush_test/hush-redir/redir_leak.right [new file with mode: 0644] blob
shell/hush_test/hush-redir/redir_leak.tests [new file with mode: 0755] blob
Unnamed repository; edit this file 'description' to name the repository.