RFR: 8258915: Temporary buffer cleanup [v2]

Weijun Wang weijun at openjdk.java.net
Fri Jan 22 14:41:10 UTC 2021


On Fri, 22 Jan 2021 07:18:22 GMT, Valerie Peng <valeriep at openjdk.org> 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



More information about the security-dev mailing list