RFR: 8282220: contentType should not be a PKCS7's member

John Jiang jjiang at openjdk.java.net
Tue Feb 22 07:30:57 UTC 2022


On Tue, 22 Feb 2022 06:36:58 GMT, Xue-Lei Andrew Fan <xuelei at openjdk.org> wrote:

>> `contentType` should be a local variable because it is only used in method `parse(DerInputStream derin, boolean oldStyle)`.
>> This change also improves some exception catch blocks.
>
> Looks like a trivial update, but please make sure to pass security tests before integration.

@XueleiFan Thanks for your review.
I run the tests in javax/net/ssl, sun/security/pkcs and sun/security/ssl, and I am waiting for the pre-submit testing results.

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

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



More information about the security-dev mailing list