commit cb05eb33692b7cfe913c338f7c8606c1edb32912
author JP Abgrall <jpa@google.com> Mon Aug 06 15:39:50 2012 -0700
committer Android (Google) Code Review <android-gerrit@google.com> Mon Aug 06 15:39:51 2012 -0700
tree a10aaec21bc907e2e12fa593d5312d69937919a1
parent 01da3942da5e0703c8aede1657f67e6adc74ccf3
parent ce4f37979316dacf99d5bd3298e870bd35d14aac

Merge "TetherController: Use correct exit() after fork()+failed exec()."

TetherController.cpp

diff --git a/TetherController.cpp b/TetherController.cpp
index 5f571a1..cb48dbc 100644
--- a/TetherController.cpp
+++ b/TetherController.cpp
@@ -158,8 +158,7 @@
             ALOGE("execl failed (%s)", strerror(errno));
         }
         ALOGE("Should never get here!");
-        free(args);
-        return 0;
+        _exit(-1);
     } else {
         close(pipefd[0]);
         mDaemonPid = pid;
@@ -233,7 +232,7 @@
         // TODO(BT) inform parent of the failure.
         //          Parent process need wait for child to report error status
         //          before it set mDhcpcdPid and return 0.
-        exit(-1);
+        _exit(-1);
     } else {
         mDhcpcdPid = pid;
         ALOGD("Reverse Tethering running, pid:%d", pid);