On Tue, 28 Jun 2022 14:49:52 GMT, Jayashree Huttanagoudar <d...@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
