commit 8e81a98a71b6ef761359acf6fcb728f7e2d3e226
author Tianjie Xu <xunchang@google.com> Fri Feb 03 22:18:50 2017 +0000
committer android-build-merger <android-build-merger@google.com> Fri Feb 03 22:18:50 2017 +0000
tree 3812ca5760f4ea1593bbf2b63d3d7d175338d6d4
parent ce7d41150d1450dbb0e883163f68267e1f169b74
parent 5eaf3cd966f76a8d6676b2b949ae97ecc505b300

Merge "Avoid to call UI functions in child process" am: 0f275ae56e am: 5fa2e576f9 am: 15ab95bb63
am: 5eaf3cd966

Change-Id: Idd211a64de13886a6ca74b5bf33159d3b90cdf7d

install.cpp

diff --git a/install.cpp b/install.cpp
index 553ebae..248a133 100644
--- a/install.cpp
+++ b/install.cpp
@@ -382,7 +382,11 @@
     umask(022);
     close(pipefd[0]);
     execv(chr_args[0], const_cast<char**>(chr_args));
-    PLOG(ERROR) << "Can't run " << chr_args[0];
+    // Bug: 34769056
+    // We shouldn't use LOG/PLOG in the forked process, since they may cause
+    // the child process to hang. This deadlock results from an improperly
+    // copied mutex in the ui functions.
+    fprintf(stdout, "E:Can't run %s (%s)\n", chr_args[0], strerror(errno));
     _exit(-1);
   }
   close(pipefd[1]);