On Wed, 24 May 2023 20:46:23 GMT, Michael McMahon <micha...@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
