Skip to content

Conversation

TimPushkin
Copy link

@TimPushkin TimPushkin commented Sep 1, 2025

Backport of JDK-8347911 for parity with Oracle's JDK 8u461.

Conflicts:

  • Copyright years
  • Missing JDK-8139206: use IOUtils.readNBytes(...) instead of the missing InputStream.readNBytes(...)

Testing: GitHub CI, jdk_imageio test group locally (headless linux/arm64)


Progress

  • Change must be properly reviewed (1 review required, with at least 1 Reviewer)
  • Change must not contain extraneous whitespace
  • Commit message must refer to an issue
  • JDK-8347911 needs maintainer approval

Integration blocker

 ⚠️ Dependency #686 must be integrated first

Issue

  • JDK-8347911: Limit the length of inflated text chunks (Bug - P3)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk8u-dev.git pull/687/head:pull/687
$ git checkout pull/687

Update a local copy of the PR:
$ git checkout pull/687
$ git pull https://git.openjdk.org/jdk8u-dev.git pull/687/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 687

View PR using the GUI difftool:
$ git pr show -t 687

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jdk8u-dev/pull/687.diff

Using Webrev

Link to Webrev Comment

Backport-of: 398a580518b4e7961bdddf733e0a89ff25bc437a
@bridgekeeper
Copy link

bridgekeeper bot commented Sep 1, 2025

👋 Welcome back tpushkin! A progress list of the required criteria for merging this PR into pr/686 will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

@openjdk
Copy link

openjdk bot commented Sep 1, 2025

❗ This change is not yet ready to be integrated.
See the Progress checklist in the description for automated requirements.

@openjdk openjdk bot changed the title Backport 398a580518b4e7961bdddf733e0a89ff25bc437a 8347911: Limit the length of inflated text chunks Sep 1, 2025
@openjdk
Copy link

openjdk bot commented Sep 1, 2025

This backport pull request has now been updated with issue from the original commit.

@openjdk openjdk bot added backport Port of a pull request already in a different code base rfr Pull request is ready for review labels Sep 1, 2025
@mlbridge
Copy link

mlbridge bot commented Sep 1, 2025

Webrevs

@TimPushkin
Copy link
Author

I believe the test failures are not related

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Port of a pull request already in a different code base rfr Pull request is ready for review
Development

Successfully merging this pull request may close these issues.

2 participants