RFR: 8321131: Console read line with zero out should zero out underlying buffer in JLine [v2]
Naoto Sato
naoto at openjdk.org
Fri Dec 1 21:24:49 UTC 2023
On Fri, 1 Dec 2023 19:40:39 GMT, Iris Clark <iris at openjdk.org> wrote:
>> Naoto Sato has updated the pull request incrementally with one additional commit since the last revision:
>>
>> moved instance fields at the top of the class
>
> src/jdk.internal.le/share/classes/jdk/internal/org/jline/JdkConsoleProviderImpl.java line 114:
>
>> 112: return null;
>> 113: } finally {
>> 114: jline.zeroOut();
>
> The location of the `jline `declaration at new line 134 is unfortunate, but not a problem this PR needs to resolve. The location makes sense if it was only used in `initJLineIfNeeded()` at some point in the past.
Thanks, Iris. Moved those instance fields to the top of the class.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16930#discussion_r1412582527
More information about the core-libs-dev
mailing list