summaryrefslogtreecommitdiffstats
path: root/unix/tclUnixPipe.c
diff options
context:
space:
mode:
authordas <das>2006-08-02 20:04:40 (GMT)
committerdas <das>2006-08-02 20:04:40 (GMT)
commit18500d2958ab58a718a10b5c7f98b2b3f3c398a9 (patch)
tree8c83f9e5ca45f3f226122aa79f28c40dbbf735ed /unix/tclUnixPipe.c
parentce0022054c0176b268c48a6bc140cec484e6cc81 (diff)
downloadtcl-18500d2958ab58a718a10b5c7f98b2b3f3c398a9.zip
tcl-18500d2958ab58a718a10b5c7f98b2b3f3c398a9.tar.gz
tcl-18500d2958ab58a718a10b5c7f98b2b3f3c398a9.tar.bz2
* unix/tclUnixPipe.c (TclpCreateProcess): for USE_VFORK: ensure standard
channels are initialized before vfork() so that the child doesn't potentially corrupt global state in the parent's address space.
Diffstat (limited to 'unix/tclUnixPipe.c')
-rw-r--r--unix/tclUnixPipe.c19
1 files changed, 18 insertions, 1 deletions
diff --git a/unix/tclUnixPipe.c b/unix/tclUnixPipe.c
index de92407..6825f24 100644
--- a/unix/tclUnixPipe.c
+++ b/unix/tclUnixPipe.c
@@ -10,7 +10,7 @@
* See the file "license.terms" for information on usage and redistribution
* of this file, and for a DISCLAIMER OF ALL WARRANTIES.
*
- * RCS: @(#) $Id: tclUnixPipe.c,v 1.23.2.6 2006/07/20 06:21:46 das Exp $
+ * RCS: @(#) $Id: tclUnixPipe.c,v 1.23.2.7 2006/08/02 20:04:40 das Exp $
*/
#include "tclInt.h"
@@ -430,6 +430,23 @@ TclpCreateProcess(interp, argc, argv, inputFile, outputFile, errorFile,
newArgv[i] = Tcl_UtfToExternalDString(NULL, argv[i], -1, &dsArray[i]);
}
+#ifdef USE_VFORK
+ /*
+ * After vfork(), do not call code in the child that changes global state,
+ * because it is using the parent's memory space at that point and writes
+ * might corrupt the parent: so ensure standard channels are initialized in
+ * the parent, otherwise SetupStdFile() might initialize them in the child.
+ */
+ if (!inputFile) {
+ Tcl_GetStdChannel(TCL_STDIN);
+ }
+ if (!outputFile) {
+ Tcl_GetStdChannel(TCL_STDOUT);
+ }
+ if (!errorFile) {
+ Tcl_GetStdChannel(TCL_STDERR);
+ }
+#endif
pid = fork();
if (pid == 0) {
int joinThisError = errorFile && (errorFile == outputFile);