RFR: 8283082: sun.security.x509.X509CertImpl.delete("x509.info.validity") nulls out info field [v2]
Jayashree Huttanagoudar
duke at openjdk.org
Wed Jun 29 16:15:33 UTC 2022
On Wed, 29 Jun 2022 13:44:11 GMT, Weijun Wang <weijun at openjdk.org> wrote:
>> Jayashree Huttanagoudar has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Address review comments
>
> src/java.base/share/classes/sun/security/x509/X509CertImpl.java line 742:
>
>> 740:
>> 741: if (id.equalsIgnoreCase(INFO)) {
>> 742: if ((attr.getSuffix() == null)) {
>
> You can remove the parentheses that were added last time.
@wangweij I have added the test case under: test/jdk/sun/security/x509/X509CertImpl/JDK8283082.java
As this is the first I am adding regression test so I am not sure whether the naming convention for the test case is right or not.
-------------
PR: https://git.openjdk.org/jdk/pull/9306
More information about the security-dev
mailing list