[jdk18] RFR: 8279222: Incorrect legacyMap.get in java.security.Provider after JDK-8276660 [v3]

Valerie Peng valeriep at openjdk.java.net
Tue Jan 4 18:10:19 UTC 2022


On Tue, 4 Jan 2022 06:59:58 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> SonarCloud reports:
>>  A "Map<ServiceKey, Service>" cannot contain a "String" in a "ServiceKey" type.
>> 
>> 
>>      // clean up old alias if present
>>      Service prevAliasService = legacyMap.get(aliasAlg);
>> 
>> 
>> Should be `aliasKey`, like other accesses to `legacyMap`. This code is introduced by [JDK-8276660](https://bugs.openjdk.java.net/browse/JDK-8276660), so it affects JDK 18.
>> 
>> Additional testing:
>>  - [x] Linux x86_64 fastdebug `jdk_security`
>
> Aleksey Shipilev has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision:
> 
>  - Whitespace
>  - Metadata updates
>  - Merge branch 'master' into JDK-8279222-jsp-get
>  - Test addition by Valerie
>  - Merge branch 'master' into JDK-8279222-jsp-get
>  - Fix

Marked as reviewed by valeriep (Reviewer).

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

PR: https://git.openjdk.java.net/jdk18/pull/70



More information about the security-dev mailing list