[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] fuzz: don't leave orphan llvm-symbolizers around
From: |
Darren Kenny |
Subject: |
Re: [PATCH] fuzz: don't leave orphan llvm-symbolizers around |
Date: |
Wed, 10 Mar 2021 11:12:50 +0000 |
On Wednesday, 2021-03-10 at 01:12:36 -05, Alexander Bulekov wrote:
> 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>
Reviewed-by: Darren Kenny <darren.kenny@oracle.com>
> ---
> tests/qtest/fuzz/generic_fuzz.c | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)
>
> diff --git a/tests/qtest/fuzz/generic_fuzz.c b/tests/qtest/fuzz/generic_fuzz.c
> index ee8c17a04c..387ae2020a 100644
> --- a/tests/qtest/fuzz/generic_fuzz.c
> +++ b/tests/qtest/fuzz/generic_fuzz.c
> @@ -583,6 +583,21 @@ static void handle_timeout(int sig)
> fprintf(stderr, "[Timeout]\n");
> 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);
> }
>
> --
> 2.28.0