RFR: 8283082: sun.security.x509.X509CertImpl.delete("x509.info.validity") nulls out info field [v2]

Jayashree Huttanagoudar duke at openjdk.org
Wed Jun 29 02:50:38 UTC 2022


On Tue, 28 Jun 2022 14:49:52 GMT, Jayashree Huttanagoudar <duke at openjdk.org> wrote:

>> The test can be much simpler than the program included in the bug report. Something like this:
>> 
>> 1. Create an empty `X509CertImpl`
>> 2. Add "x509.info"
>> 3. Add "x509.info.issuer"
>> 4. Remove "x509.info.issuer"
>> 5. Add "x509.info.issuer" again
>> 
>> Without the fix, step 4 removes the whole info and step 5 would fail.
>
> Thank you for taking a look at the PR.
> Sure I will address these comments.

> The test can be much simpler than the program included in the bug report. Something like this:
> 
>     1. Create an empty `X509CertImpl`
> 
>     2. Add "x509.info"
> 
>     3. Add "x509.info.issuer"
> 
>     4. Remove "x509.info.issuer"
> 
>     5. Add "x509.info.issuer" again
> 
> 
> Without the fix, step 4 removes the whole info and step 5 would fail.
I am not a getting a clear idea about how to add issuer and delete etc. 
Could you please point me to some example?

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

PR: https://git.openjdk.org/jdk/pull/9306



More information about the security-dev mailing list