[jdk11u-dev] RFR: 8239264: Clearup the legacy ObjectIdentifier constructor from int array [v3]

[Goetz Lindenmaier]

On Mon, 29 May 2023 10:22:14 GMT, Goetz Lindenmaier <goetz at openjdk.org> wrote:

>> I had to work on this intensely to get it into 11.
>> 
>> First I resolved the conflicts:
>> 
>> src/java.base/macosx/classes/apple/security/KeychainStore.java
>> Copyright, different way to call loadLibrary.
>> This was wrongly removed anyways, see JDK-8240193
>> 
>> src/java.base/share/classes/com/sun/crypto/provider/DHPrivateKey.java
>> src/java.base/share/classes/com/sun/crypto/provider/DHPublicKey.java
>> src/java.base/share/classes/com/sun/crypto/provider/OAEPParameters.java
>> src/java.base/share/classes/sun/security/pkcs/ContentInfo.java
>> All Copyright
>> 
>> src/java.base/share/classes/sun/security/pkcs/PKCS9Attribute.java
>> The change is subsumed by 8268801: Improve PKCS attribute handling", a
>> closed issue that is not in head.
>> 8268801 brought parts of this change to
>> 11 as it was needed for some security fix.
>> 
>> src/java.base/share/classes/sun/security/pkcs12/PKCS12KeyStore.java
>> Copyright, Complex resolve due to context
>> 
>> src/java.base/share/classes/sun/security/util/ObjectIdentifier.java
>> Copyright
>> Complex resolve due to context.  Hunk in line 486 actually differs, but change
>> is only new formatting.
>> 
>> src/java.base/share/classes/sun/security/x509/AlgorithmId.java
>> Copyright
>> 
>> Second, to make it compile, I had to adapt the newly introduced calls to
>> ObjectIdentifier.of(). Unfortunately, "8268801: Improve PKCS attribute handling"
>> already backported parts of the later change
>> "8242151: Improve OID mapping and reuse among JDK security providers f..."
>> that make of() throw an exception. I restored the original function,
>> renamed it and adapted all the call sites.
>> 
>> This comes in a separate commit.  I will backport "JDK-8242151: Improve OID mapping and reuse among JDK security providers for aliases registration" on top of this, which will remove of1() again.
>
> Goetz Lindenmaier has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:
> 
>   Add back newInternal()

@martinuy, I know you are an expert on these files, I would appreciate a review of this backport by you, as well as
#1908 and maybe #1909 . Thanks a lot!

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

PR Comment: https://git.openjdk.org/jdk11u-dev/pull/1898#issuecomment-1573793412