commit 1d5d475b514bf7f291fc394b50a1784045d39d07
author Paul Crowley <paulcrowley@google.com> Fri Dec 07 18:15:37 2018 -0800
committer android-build-merger <android-build-merger@google.com> Fri Dec 07 18:15:37 2018 -0800
tree c3ea4eddbdd3aec67a3c51ebda0e9ab2c5498639
parent f0c43617704a964bbb30d56689e10adac0da4f5a
parent b1166633b69e0277cfe2d8ffe1413684c4798bf1

Merge "Check for errors in dup2"
am: b1166633b6

Change-Id: I1b0144fbf19892012baebe08bbaab9f7e77bf71e

Utils.cpp

diff --git a/Utils.cpp b/Utils.cpp
index 3bbdf3a..a77b1b6 100644
--- a/Utils.cpp
+++ b/Utils.cpp
@@ -312,7 +312,10 @@
             }
         }
         pipe_read.reset();
-        dup2(pipe_write.get(), STDOUT_FILENO);
+        if (dup2(pipe_write.get(), STDOUT_FILENO) == -1) {
+            PLOG(ERROR) << "dup2 in ForkExecvp";
+            _exit(EXIT_FAILURE);
+        }
         pipe_write.reset();
         execvp(argv[0], const_cast<char**>(argv.data()));
         PLOG(ERROR) << "exec in ForkExecvp";