Integrated: JDK-8321599 Data loss in AVX3 Base64 decoding
Scott Gibbons
sgibbons at openjdk.org
Thu Jan 4 01:39:39 UTC 2024
On Fri, 8 Dec 2023 20:56:52 GMT, Scott Gibbons <sgibbons at openjdk.org> wrote:
> Fix for looking for padding characters within the encoded string. Was not adding start offset to length, so was looking at potentially freed or uninitialized memory.
>
> Tested teir1 and with testcase supplied with JBS issue.
>
> The problem will only occur when all of the following are true:
> 1. The source offset of the string to be decoded is != 0.
> 2. The characters at the beginning of the string (minus the offset) plus the string length mod 64 are either "=" or "==".
> 3. The string is >= 32 characters.
> 4. The string is not MIME encoded.
>
> If any of these conditions are not met, the decode works as expected. This was due to omitting the source offset of the string when checking for padding characters.
This pull request has now been integrated.
Changeset: 13c11487
Author: Scott Gibbons <sgibbons at openjdk.org>
Committer: Sandhya Viswanathan <sviswanathan at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/13c11487f7126a370d9ce8e62f661ea83eedefe6
Stats: 124 lines in 2 files changed: 121 ins; 0 del; 3 mod
8321599: Data loss in AVX3 Base64 decoding
Reviewed-by: sviswanathan, kvn
-------------
PR: https://git.openjdk.org/jdk/pull/17039
More information about the hotspot-compiler-dev
mailing list