From 29fc5b76cdfa2f32e7965f54aeb28356b93f3704 Mon Sep 17 00:00:00 2001 From: Michael Pratt Date: Fri, 26 May 2017 16:11:19 -0700 Subject: 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. --- executor/executor.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'executor') 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() -- cgit mrf-deployment