On Fri, 22 Jan 2021 07:18:22 GMT, Valerie Peng <[email protected]> wrote:
>> Weijun Wang has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains three commits:
>>
>> - rsa
>> - Merge
>> - 8258915: More temporary buffer cleanup
>>
>> 8258915: More temporary buffer cleanup
>
> src/java.base/share/classes/com/sun/crypto/provider/PrivateKeyInfo.java line
> 98:
>
>> 96:
>> 97: public void clear() {
>> 98: Arrays.fill(privkey, (byte)0);
>
> check for null just in case?
`val.data.getOctetString()` on line 82 should never return null, so I think a
null check is not necessary here.
-------------
PR: https://git.openjdk.java.net/jdk/pull/2070
