On Fri, Oct 18, 2024 at 11:23 AM Christian Göttsche <cgoettsche@xxxxxxxxxxxxx> wrote: > > From: Christian Göttsche <cgzones@xxxxxxxxxxxxxx> > > setjmp(3) returns 0 on the first fake invocation, adjust the condition > accordingly. > > Reported by the OSS Fuzz Introspector[1]. > > [1]: https://storage.googleapis.com/oss-fuzz-introspector/selinux/inspector-report/20241016/fuzz_report.html > > Fixes: f07fc2a75 ("checkpolicy/fuzz: override YY_FATAL_ERROR") > Signed-off-by: Christian Göttsche <cgzones@xxxxxxxxxxxxxx> Acked-by: James Carter <jwcart2@xxxxxxxxx> > --- > checkpolicy/fuzz/checkpolicy-fuzzer.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/checkpolicy/fuzz/checkpolicy-fuzzer.c b/checkpolicy/fuzz/checkpolicy-fuzzer.c > index ddb43260..331201c0 100644 > --- a/checkpolicy/fuzz/checkpolicy-fuzzer.c > +++ b/checkpolicy/fuzz/checkpolicy-fuzzer.c > @@ -101,7 +101,7 @@ static int read_source_policy(policydb_t *p, const uint8_t *data, size_t size) > > init_parser(1); > > - if (!setjmp(fuzzing_pre_parse_stack_state)) { > + if (setjmp(fuzzing_pre_parse_stack_state) != 0) { > queue_destroy(id_queue); > fclose(yyin); > yylex_destroy(); > -- > 2.45.2 > >
