Apache POI: install libraries for awt and adjust xmx #11006
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The UnsatisfiedLinkError keeps us from covering quite some code, so let's try to fetch required .so files and include them in the container image so that some basic AWT classes like "Color" can be used. Reading details of fonts likely still does not work, but Apache POI tries to gracefully handle this case already. See related discussions at google#7380 and google#10933
Some tests currently fail with a native-OOM, it seems this happens if the Java-Xmx-setting is too high so that Jazzer does not get enough room for its data. Therefore we reduce the Xmx from 2048m to 1024m so that Jazzer has enough native memory available. Xmx up to 1500m still caused OOMs in local testing. This should fix requent OOMs in oss-fuzz, e.g. https://oss-fuzz.com/testcase-detail/4597575322566656
|
centic9 has previously contributed to projects/apache-poi. The previous PR was #10978 |
DavidKorczynski
approved these changes
Sep 24, 2023
This was referenced Sep 25, 2023
gedigi
pushed a commit
to gedigi/oss-fuzz
that referenced
this pull request
Sep 27, 2023
These changes add some additional .so files to the container so that loading some classes from java.awt works. See google#10933 and google#7380 for related discussions. Also decrease "Xmx" as it seems we get native OOM in Jazzer-runs otherwise.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
These changes add some additional .so files to the container so that loading some classes from java.awt works.
See #10933 and #7380 for related discussions.
Also decrease "Xmx" as it seems we get native OOM in Jazzer-runs otherwise.