summaryrefslogtreecommitdiff
path: root/src/stdio/popen.c
diff options
context:
space:
mode:
authorRich Felker <dalias@aerifal.cx>2012-06-20 14:39:50 -0400
committerRich Felker <dalias@aerifal.cx>2012-06-20 14:39:50 -0400
commit9c21f4342c787e04a9e31ad8a1d87a65c89968ca (patch)
treeec09299484f96dd61e6ebe84f37ceab69610a3f5 /src/stdio/popen.c
parentf305467aad4659e92fff84be6c89d4767b6f99d4 (diff)
downloadmusl-9c21f4342c787e04a9e31ad8a1d87a65c89968ca.tar.gz
musl-9c21f4342c787e04a9e31ad8a1d87a65c89968ca.tar.bz2
musl-9c21f4342c787e04a9e31ad8a1d87a65c89968ca.tar.xz
musl-9c21f4342c787e04a9e31ad8a1d87a65c89968ca.zip
make popen cancellation-safe
close was the only cancellation point called from popen, but it left popen with major resource leaks if any call to close got cancelled. the easiest, cheapest fix is just to use a non-cancellable close function.
Diffstat (limited to 'src/stdio/popen.c')
-rw-r--r--src/stdio/popen.c7
1 files changed, 7 insertions, 0 deletions
diff --git a/src/stdio/popen.c b/src/stdio/popen.c
index 50765daa..4f9d6e9e 100644
--- a/src/stdio/popen.c
+++ b/src/stdio/popen.c
@@ -1,4 +1,11 @@
#include "stdio_impl.h"
+#include "syscall.h"
+
+static inline void nc_close(int fd)
+{
+ __syscall(SYS_close, fd);
+}
+#define close(x) nc_close(x)
FILE *popen(const char *cmd, const char *mode)
{