jdk8u-dev icon indicating copy to clipboard operation
jdk8u-dev copied to clipboard

Published 20 hours ago • verified publisher icon openjdk

8347911: Limit the length of inflated text chunks

Open TimPushkin opened this issue 3 months ago • 11 comments

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)
  • [x] Change must not contain extraneous whitespace
  • [x] 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

TimPushkin avatar Sep 01 '25 07:09 TimPushkin

:wave: 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.

bridgekeeper[bot] avatar Sep 01 '25 07:09 bridgekeeper[bot]

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

openjdk[bot] avatar Sep 01 '25 07:09 openjdk[bot]

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

openjdk[bot] avatar Sep 01 '25 07:09 openjdk[bot]

Webrevs

mlbridge[bot] avatar Sep 01 '25 07:09 mlbridge[bot]

I believe the test failures are not related

TimPushkin avatar Sep 01 '25 14:09 TimPushkin

@TimPushkin This pull request has been inactive for more than 4 weeks and will be automatically closed if another 4 weeks passes without any activity. To avoid this, simply issue a /touch or /keepalive command to the pull request. Feel free to ask for assistance if you need help with progressing this pull request towards integration!

bridgekeeper[bot] avatar Sep 30 '25 09:09 bridgekeeper[bot]

/touch

TimPushkin avatar Sep 30 '25 09:09 TimPushkin

@TimPushkin The pull request is being re-evaluated and the inactivity timeout has been reset.

openjdk[bot] avatar Sep 30 '25 09:09 openjdk[bot]

@TimPushkin This pull request has been inactive for more than 4 weeks and will be automatically closed if another 4 weeks passes without any activity. To avoid this, simply issue a /touch or /keepalive command to the pull request. Feel free to ask for assistance if you need help with progressing this pull request towards integration!

bridgekeeper[bot] avatar Oct 28 '25 14:10 bridgekeeper[bot]

/touch

TimPushkin avatar Oct 28 '25 15:10 TimPushkin

@TimPushkin The pull request is being re-evaluated and the inactivity timeout has been reset.

openjdk[bot] avatar Oct 28 '25 15:10 openjdk[bot]

@TimPushkin This pull request has been inactive for more than 4 weeks and will be automatically closed if another 4 weeks passes without any activity. To avoid this, simply issue a /touch or /keepalive command to the pull request. Feel free to ask for assistance if you need help with progressing this pull request towards integration!

bridgekeeper[bot] avatar Nov 26 '25 05:11 bridgekeeper[bot]

/touch

TimPushkin avatar Nov 26 '25 06:11 TimPushkin

@TimPushkin The pull request is being re-evaluated and the inactivity timeout has been reset.

openjdk[bot] avatar Nov 26 '25 06:11 openjdk[bot]