diff options
| author | Tianjie Xu <xunchang@google.com> | 2017-02-03 23:16:19 +0100 |
|---|---|---|
| committer | android-build-merger <android-build-merger@google.com> | 2017-02-03 23:16:19 +0100 |
| commit | 5eaf3cd966f76a8d6676b2b949ae97ecc505b300 (patch) | |
| tree | 9a763722edac743ffc04fd1b8781b647f3a65fd8 | |
| parent | Merge "update_verifier should read dm wrapped partition" am: c44c479197 am: d40cd0e7cf (diff) | |
| parent | Merge "Avoid to call UI functions in child process" am: 0f275ae56e (diff) | |
| download | android_bootable_recovery-5eaf3cd966f76a8d6676b2b949ae97ecc505b300.tar android_bootable_recovery-5eaf3cd966f76a8d6676b2b949ae97ecc505b300.tar.gz android_bootable_recovery-5eaf3cd966f76a8d6676b2b949ae97ecc505b300.tar.bz2 android_bootable_recovery-5eaf3cd966f76a8d6676b2b949ae97ecc505b300.tar.lz android_bootable_recovery-5eaf3cd966f76a8d6676b2b949ae97ecc505b300.tar.xz android_bootable_recovery-5eaf3cd966f76a8d6676b2b949ae97ecc505b300.tar.zst android_bootable_recovery-5eaf3cd966f76a8d6676b2b949ae97ecc505b300.zip | |
| -rw-r--r-- | install.cpp | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/install.cpp b/install.cpp index 553ebae10..248a1336e 100644 --- a/install.cpp +++ b/install.cpp @@ -382,7 +382,11 @@ static int try_update_binary(const char* path, ZipArchiveHandle zip, bool* wipe_ 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]); |