fuzz: don't leave orphan llvm-symbolizers around
I noticed that with a sufficiently small timeout, the fuzzer fork-server sometimes locks up. On closer inspection, the issue appeared to be caused by entering our SIGALRM handler, while libfuzzer is in it's crash handlers. Because libfuzzer relies on pipe communication with an external child process to print out stack-traces, we shouldn't exit early, and leave an orphan child. Check for children in the SIGALRM handler to avoid this issue. Signed-off-by: Alexander Bulekov <alxndr@bu.edu> Acked-by: Thomas Huth <thuth@redhat.com> Reviewed-by: Darren Kenny <darren.kenny@oracle.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
This commit is contained in:
parent
e8a2a62106
commit
d0614b8e7a
|
@ -583,6 +583,21 @@ static void handle_timeout(int sig)
|
||||||
fprintf(stderr, "[Timeout]\n");
|
fprintf(stderr, "[Timeout]\n");
|
||||||
fflush(stderr);
|
fflush(stderr);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/*
|
||||||
|
* If there is a crash, libfuzzer/ASAN forks a child to run an
|
||||||
|
* "llvm-symbolizer" process for printing out a pretty stacktrace. It
|
||||||
|
* communicates with this child using a pipe. If we timeout+Exit, while
|
||||||
|
* libfuzzer is still communicating with the llvm-symbolizer child, we will
|
||||||
|
* be left with an orphan llvm-symbolizer process. Sometimes, this appears
|
||||||
|
* to lead to a deadlock in the forkserver. Use waitpid to check if there
|
||||||
|
* are any waitable children. If so, exit out of the signal-handler, and
|
||||||
|
* let libfuzzer finish communicating with the child, and exit, on its own.
|
||||||
|
*/
|
||||||
|
if (waitpid(-1, NULL, WNOHANG) == 0) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
_Exit(0);
|
_Exit(0);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue