From be857bfd9b96f8069d1a2a5447ab8ae922722716 Mon Sep 17 00:00:00 2001
From: Paul Crowley <paulcrowley@google.com>
Date: Fri, 7 Dec 2018 12:23:25 -0800
Subject: [PATCH] Check for errors in dup2

Bug: 26735063
Test: adb shell sm partition disk:7,3 private && adb logcat -d
Change-Id: I3aa8d1f6183dd5e77f54f422482a54ea8197d768
---
 Utils.cpp | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/Utils.cpp b/Utils.cpp
index 3bbdf3a..a77b1b6 100644
--- a/Utils.cpp
+++ b/Utils.cpp
@@ -312,7 +312,10 @@ status_t ForkExecvp(const std::vector<std::string>& args, std::vector<std::strin
             }
         }
         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";