RFR: 8300038: Make new version of JNU_GetStringPlatformChars which checks for null characters [v2]
Daniel Fuchs
dfuchs at openjdk.org
Tue May 23 15:31:51 UTC 2023
On Tue, 23 May 2023 15:18:36 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 incrementally with one additional commit since the last revision:
>
> test comment update
test/jdk/java/net/InetAddress/NullCharDriver.java line 29:
> 27: * @modules java.base/java.net
> 28: * @compile/module=java.base java/net/NullChar.java
> 29: * @summary foo
It would be good to have a better summary ;-)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14083#discussion_r1202541587
More information about the core-libs-dev
mailing list