Merge "adbd's children should have SIGPIPE set to SIG_DFL."

This commit is contained in:
Treehugger Robot 2017-02-17 20:55:54 +00:00 committed by Gerrit Code Review
commit 81c6ff3b96

View file

@ -320,6 +320,10 @@ bool Subprocess::ForkAndExec(std::string* error) {
parent_error_sfd.reset(-1);
close_on_exec(child_error_sfd);
// adbd sets SIGPIPE to SIG_IGN to get EPIPE instead, and Linux propagates that to child
// processes, so we need to manually reset back to SIG_DFL here (http://b/35209888).
signal(SIGPIPE, SIG_DFL);
if (command_.empty()) {
execle(_PATH_BSHELL, _PATH_BSHELL, "-", nullptr, cenv.data());
} else {