RFR: 8272613: CharsetDecoder.decode(ByteBuffer) throws IllegalArgumentException

Alan Bateman alanb at openjdk.org
Mon Mar 27 12:58:29 UTC 2023


On Tue, 21 Mar 2023 22:40:48 GMT, Naoto Sato <naoto at openjdk.org> wrote:

> Addressing an integer overflow on decoding a large byte buffer. Now that the code only throws OOME if necessary, describing the fact with the `@throws` tag. A CSR for that is also drafted.

This change looks okay but I assume the test won't run on 32-bit due to the array size. It might need `@requires sun.arch.data.model == "64"`.

This change looks okay but I assume the test won't run on 32-bit due to the array size. It might need `@requires sun.arch.data.model == "64"`.

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

PR Comment: https://git.openjdk.org/jdk/pull/13131#issuecomment-1485065367
PR Comment: https://git.openjdk.org/jdk/pull/13131#issuecomment-1485065518


More information about the nio-dev mailing list