commit c8dc63b14f5bcef34a23061c17d3bbe66df51429
author JP Abgrall <jpa@google.com> Wed Feb 13 16:30:00 2013 -0800
committer JP Abgrall <jpa@google.com> Wed Feb 13 16:30:00 2013 -0800
tree 288bb0eea55de7b03cf02caad7cedf183eb2851b
parent 001f0a436e9fe0353dccd98ee34b91095d9ed1a1

BandwidthController: correctly handle the iptables command exit code

The return code from android_fork_execvp() needed to be massaged because
a status ptr was passed in.

Bug: 8185095
Change-Id: I205884f09b563dd729423a74f22fd5b8c2d59c64

BandwidthController.cpp

diff --git a/BandwidthController.cpp b/BandwidthController.cpp
index 8f3a738..e4171d6 100644
--- a/BandwidthController.cpp
+++ b/BandwidthController.cpp
@@ -198,11 +198,10 @@
     argv[argc] = NULL;
     res = android_fork_execvp(argc, (char **)argv, &status, false,
             failureHandling == IptFailShow);
-
-    if ((res || !WIFEXITED(status) || WEXITSTATUS(status)) &&
-            failureHandling == IptFailShow) {
-        ALOGE("runIptablesCmd(): failed %s res=%d status=%d", fullCmd.c_str(),
-                res, status);
+    res = res || !WIFEXITED(status) || WEXITSTATUS(status);
+    if (res && failureHandling == IptFailShow) {
+      ALOGE("runIptablesCmd(): res=%d status=%d failed %s", res, status,
+            fullCmd.c_str());
     }
     return res;
 }