Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update jazzer #181

Merged
merged 5 commits into from
Jan 3, 2022
Merged

Update jazzer #181

merged 5 commits into from
Jan 3, 2022

Conversation

fmeum
Copy link
Member

@fmeum fmeum commented Oct 22, 2021 •
edited
Loading

This brings improved macOS support as well as a configurable deps for java_fuzz_test.

@fmeum fmeum force-pushed the update-jazzer branch 4 times, most recently from e846955 to 217703f Compare October 22, 2021 14:20
@fmeum fmeum marked this pull request as ready for review October 22, 2021 15:01
@fmeum fmeum marked this pull request as draft October 23, 2021 11:38
@fmeum
Copy link
Member Author

fmeum commented Oct 23, 2021

@stefanbucur You can already review this. I just want to bump some Jazzer dependencies.

If you prefer, I could also make the deps argument of cc_fuzz_test configurable in the same way.

@fmeum
Copy link
Member Author

fmeum commented Dec 30, 2021

I updated Jazzer to a version with native support for M1 Macs. I don't know whether Honggfuzz supports it, but everything else in rules_fuzzing should.

@fmeum fmeum marked this pull request as ready for review December 30, 2021 20:35
@fmeum fmeum marked this pull request as draft December 31, 2021 07:45
@fmeum fmeum marked this pull request as ready for review December 31, 2021 13:31
fmeum added 5 commits January 3, 2022 17:18
@fmeum
Update Jazzer
9929157 
The new Jazzer version includes logic to find libjvm.so, improved
instrumentation for the Map interface as well as support for M1 Macs.
@fmeum
Symlink missing local_jdk runfiles in OSS-Fuzz
bcb2786
@fmeum
Create Java native libraries with .dylib extension on macOS
8136d31
@fmeum
Test Java examples in the CI on macOS
ce3bc2f 
The newest Jazzer version finds libjvm.dylib on macOS.
@fmeum
Add more comments to java_fuzz_test examples
8cecd95
stefanbucur
stefanbucur approved these changes Jan 3, 2022
View reviewed changes
examples/java/BUILD Show resolved Hide resolved
@stefanbucur stefanbucur merged commit 9b3d9a8 into bazel-contrib:master Jan 3, 2022
@fmeum fmeum deleted the update-jazzer branch January 3, 2022 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stefanbucur stefanbucur stefanbucur approved these changes

@asraa asraa Awaiting requested review from asraa asraa is a code owner

@inferno-chromium inferno-chromium Awaiting requested review from inferno-chromium inferno-chromium is a code owner

@markus-kusano markus-kusano Awaiting requested review from markus-kusano

@oliverchang oliverchang Awaiting requested review from oliverchang oliverchang is a code owner

@xinhaoyuan xinhaoyuan Awaiting requested review from xinhaoyuan xinhaoyuan is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fmeum @stefanbucur