Merge "adbd: fix spurious close of uninitialized fd."

This commit is contained in:
Josh Gao 2016-03-05 03:04:36 +00:00 committed by Gerrit Code Review
commit 3c608c7425

View file

@ -315,7 +315,9 @@ bool Subprocess::ForkAndExec(std::string* error) {
if (type_ == SubprocessType::kPty) {
int fd;
pid_ = forkpty(&fd, pts_name, nullptr, nullptr);
stdinout_sfd_.Reset(fd);
if (pid_ > 0) {
stdinout_sfd_.Reset(fd);
}
} else {
if (!CreateSocketpair(&stdinout_sfd_, &child_stdinout_sfd)) {
*error = android::base::StringPrintf("failed to create socketpair for stdin/out: %s",