RFR: 8246797: A convenient method to read OPTIONAL element [v2]

Valerie Peng valeriep at openjdk.java.net
Tue Aug 24 17:07:30 UTC 2021


On Tue, 24 Aug 2021 14:24:00 GMT, Weijun Wang <weijun at openjdk.org> wrote:

>> This code change adds new methods to `DerInputStream` to easily and safely read optional fields in a ASN.1 DER-encoded value. It also adds several wrapping methods to `DerValue` to avoid unnecessary memory copying when calling by an internal method.
>> 
>> The new methods are applied to `OAEPParameters` to show out how they work. A new regression test is added to show that not only the new methods are simper but they are safer and more correct.
>
> Weijun Wang has updated the pull request incrementally with one additional commit since the last revision:
> 
>   best practice for reading DEFAULT, switch expression, etc

src/java.base/share/classes/sun/security/util/DerValue.java line 306:

> 304: 
> 305:     /**
> 306:      * Wraps a byte array at a single DerValue.

nit: at=>as

src/java.base/share/classes/sun/security/util/DerValue.java line 319:

> 317: 
> 318:     /**
> 319:      * Wraps a byte array at a single DerValue.

nit: at=>as

-------------

PR: https://git.openjdk.java.net/jdk/pull/5221



More information about the security-dev mailing list