aboutsummaryrefslogtreecommitdiffstats
path: root/executor
diff options
context:
space:
mode:
authorMichael Pratt <mpratt@google.com>2017-05-26 16:11:19 -0700
committerMichael Pratt <mpratt@google.com>2017-05-30 09:33:51 -0700
commit29fc5b76cdfa2f32e7965f54aeb28356b93f3704 (patch)
tree28d1ae8bd5bc68a62ab219b609c257866fcceca4 /executor
parent75b66eabef6ee2b89d89f31d90ff2ed9d3478c39 (diff)
all: cleanup executor/ipc status checking
This is mostly a cleanup change with little functional change. In ipc.command.exec, remove the status fallback from the pipe to the exit status. Once the executor is serving, it always writes the status over the pipe; anything else is an error. Remove the panic check in syz-stress, which is no longer needed.
Diffstat (limited to 'executor')
-rw-r--r--executor/executor.cc3
1 files changed, 2 insertions, 1 deletions
diff --git a/executor/executor.cc b/executor/executor.cc
index cb31e0697..044410792 100644
--- a/executor/executor.cc
+++ b/executor/executor.cc
@@ -226,7 +226,8 @@ int main(int argc, char** argv)
// ptrace(PTRACE_SEIZE, 1, 0, 0x100040)
// This is unfortunate, but I don't have a better solution than ignoring it for now.
exitf("loop exited with status %d", status);
- return 0;
+ // Unreachable.
+ return 1;
}
void loop()