RFR: 6478546: FileInputStream.read() throws OutOfMemoryError when there is plenty available [v6]

Brian Burkhalter bpb at openjdk.org
Mon Nov 6 22:36:50 UTC 2023


> Limit native memory allocation and move write loop from the native layer into Java. This change should make the OOME reported in the issue much less likely.

Brian Burkhalter has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains eight additional commits since the last revision:

 - Merge
 - Merge
 - 6478546: do-while -> while-do in writeBytes; remove NULL and bounds checks in native
 - 6478546: Move buffer clamping up to Java layer; correct read behavior to match legacy
 - 6478546: Decrease malloc limit to 1.5 MB
 - 6478546: Minor refactoring
 - 6478546: Prevent short read
 - 6478546: FileInputStream.read() throws OutOfMemoryError when there is plenty available

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/14981/files
  - new: https://git.openjdk.org/jdk/pull/14981/files/3a016a72..408e2df6

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=14981&range=05
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=14981&range=04-05

  Stats: 69129 lines in 2383 files changed: 38779 ins; 14073 del; 16277 mod
  Patch: https://git.openjdk.org/jdk/pull/14981.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/14981/head:pull/14981

PR: https://git.openjdk.org/jdk/pull/14981


More information about the security-dev mailing list