[Tarantool-patches] [PATCH 1/4] test: add infrastructure for fuzzing testing and fuzzers
Igor Munkin
imun at tarantool.org
Thu Dec 24 20:50:34 MSK 2020
Sergey,
On 24.12.20, Sergey Bronnikov wrote:
> Igor,
>
<snipped>
> >>>>> 2. Do you need to specify <address> flag once more, when ASAN is
> >>>>> enabled? If not the hunk above looks excess, doesn't it?
> >>>> Agree, it was a bad idea to manage UBSan and ASAN flags in yet another
> >>>> place.
> > I guess this should be fixed in scope of the first patch, but I see you
> > squashed it to the last one. Why?
>
> It seems because I was triggered by OSS_FUZZ in hunk and squashed it to
> a commit
>
> that introduce an OSS_FUZZ support. Fixed it and also moved
> add_compile_options() to the first commit too
>
> (otherwise project source code is not instrumented with
> -fsanitize=fuzzer-no-link).
OK, but I believe we agreed the <address> options should be removed to
avoid managing ASAN flags here, didn't we?
>
<snipped>
--
Best regards,
IM
More information about the Tarantool-patches
mailing list