oss-fuzz icon indicating copy to clipboard operation
oss-fuzz copied to clipboard

infra: Fix possible bugs in jvm coverage report generation

Open arthurscchan opened this issue 1 year ago • 3 comments

In #10906, the logic has skipped the removal of duplication for the fuzzer classes in the base directory because fuzzer classes don't have duplication in the jar files. But in some projects, fuzzer classes could also contain duplicates from different fuzzers if they share some common base classes. In this situation, the duplication of the fuzzer classes will still crash the Jacoco report generation and merging. This PR fixes that by reverting the changes in #10906 to remove all duplicate classes first. Then an additional line of logic is added to copy the fuzzer classes from the $OUT directory to ensure only one copy of the same fuzzer class exists in the dump directory. This could avoid duplication of fuzzer-related classes that crash the Jacoco report generation process.

arthurscchan avatar Jan 30 '24 07:01 arthurscchan

/gcbrun trial_build.py java --fuzzing-engines libfuzzer --sanitizers coverage

jonathanmetzman avatar Feb 21 '24 16:02 jonathanmetzman

Is this PR still relevant?

jonathanmetzman avatar May 02 '24 15:05 jonathanmetzman

/gcbrun trial_build.py jvm --fuzzing-engines libfuzzer --sanitizers coverage

jonathanmetzman avatar May 02 '24 15:05 jonathanmetzman