RFR: 8300038: Make new version of JNU_GetStringPlatformChars which checks for null characters [v4]

Michael McMahon michaelm at openjdk.org
Thu May 25 14:23:58 UTC 2023


On Wed, 24 May 2023 20:46:23 GMT, Michael McMahon <michaelm at openjdk.org> wrote:

>> This PR creates a new version of the JNI utility function JNU_GetStringPlatformChars called JNU_GetStringPlatformCharsStrict, which performs additional validation of the returned string, namely that it does not contain any embedded NULL characters. If any such characters are found the function returns NULL with an IAE pending. The change also switches usage in the networking native code to use the new function.
>> 
>> This cautious approach was taken rather than changing the behavior of the existing function as each native code area needs to review the effect of making the switch. Otherwise, surprising behavior changes might occur (eg undocumented IAE being thrown to user code instead of some other exception).
>
> Michael McMahon has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 12 commits:
> 
>  - Merge branch 'master' into nullStrings
>  - error message and test update
>  - Merge branch 'master' into nullStrings
>  - test comment update
>  - test update
>  - Merge branch 'master' into nullStrings
>  - exception message update
>  - test update
>  - remve whitespace
>  - update
>  - ... and 2 more: https://git.openjdk.org/jdk/compare/207fbcb0...35df1a67

I'll integrate this today unless there are further comments.

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

PR Comment: https://git.openjdk.org/jdk/pull/14083#issuecomment-1562998991


More information about the core-libs-dev mailing list