> DTLSReassembler can queue fragments of the same handshake message with > different total messageLength values. Reassembly holes are initialized from > the first fragment, while the output array is allocated using the fragment > with the lowest offset. > > For the same epoch, message sequence, and HELLO_REQUEST type, submit these > fragments in order: > offset 5, length 50, total message length 100; > offset 0, length 5, total message length 10; > offset 55, length 45, total message length 100. > > The final fragment completes the holes, but copying the buffered fragments > into the array sized for a ten-byte message triggers > ArrayIndexOutOfBoundsException. SSLEngineImpl catches this runtime exception > and reports a fatal SSLException; it does not establish a JVM crash. > > Expected behavior the inconsistent handshake metadata must be rejected as a > protocol error before fragment reassembly. > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai).
Timofei Fedotov has updated the pull request incrementally with two additional commits since the last revision: - Verify complete DTLS handshake fragment reassembly - Test inconsistent DTLS handshake fragment types ------------- Changes: - all: https://git.openjdk.org/jdk/pull/33633/files - new: https://git.openjdk.org/jdk/pull/33633/files/9df8ebfa..85d1208f Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=33633&range=06 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=33633&range=05-06 Stats: 97 lines in 2 files changed: 65 ins; 5 del; 27 mod Patch: https://git.openjdk.org/jdk/pull/33633.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/33633/head:pull/33633 PR: https://git.openjdk.org/jdk/pull/33633
