On Thu, 18 Aug 2022 10:03:23 GMT, Matthias Baesken <mbaes...@openjdk.org> wrote:
> NTLMAuthSequence getCredentialsHandle has a number of cleanup paths doing
> various JNU_ReleaseStringPlatformChars calls; those could be placed at one
> place in the function.
Looks good. Minor suggestion that the check in line 144 be changed to:
if (ss == SEC_E_OK) {
which complements the change to the initialization of variable ss.
-------------
Changes requested by michaelm (Reviewer).
PR: https://git.openjdk.org/jdk/pull/9914
