From 339685269f8bec31eeb0edb98c832dde277c9c7d Mon Sep 17 00:00:00 2001 From: Dan Gohman Date: Wed, 5 Aug 2009 00:09:12 +0000 Subject: [PATCH] Use _exit rather than exit in the child process after a failed exec. Add a comment explaining why. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@78128 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/System/Unix/Program.inc | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/lib/System/Unix/Program.inc b/lib/System/Unix/Program.inc index 4814228e138..43a3e7f2833 100644 --- a/lib/System/Unix/Program.inc +++ b/lib/System/Unix/Program.inc @@ -200,9 +200,13 @@ Program::Execute(const Path& path, execve(path.c_str(), (char**)args, (char**)envp); else execv(path.c_str(), (char**)args); - // If the execve() failed, we should exit and let the parent pick up - // our non-zero exit status. - exit(errno == ENOENT ? 127 : 126); + // If the execve() failed, we should exit. Follow Unix protocol and + // return 127 if the executable was not found, and 126 otherwise. + // Use _exit rather than exit so that atexit functions and static + // object destructors cloned from the parent process aren't + // redundantly run, and so that any data buffered in stdio buffers + // cloned from the parent aren't redundantly written out. + _exit(errno == ENOENT ? 127 : 126); } // Parent process: Break out of the switch to do our processing. -- 2.34.1