[jdk11u-dev] RFR: 8321925: sun/security/mscapi/KeytoolChangeAlias.java fails with "Alias <246810> does not exist" [v2]

[Martin Doerr]

On Mon, 13 May 2024 09:38:10 GMT, SendaoYan <syan at openjdk.org> wrote:

>> Hi, 
>> 
>> Backport of [JDK-8321925](https://bugs.openjdk.org/browse/JDK-8321925). 
>> 
>> The origin PR use `int nextInt(int origin, int bound)` [link](https://docs.oracle.com/en/java/javase/17/docs/api/java.base/java/util/random/RandomGenerator.html#nextInt(int,int)) , but the jdk11 do not have this API. So I change `int nextInt(int origin, int bound)` to `nextInt(bound - origin) + origin` [link](https://docs.oracle.com/en/java/javase/11/docs/api/java.base/java/util/Random.html#nextInt(int)).
>> 
>> Only change the testcase, the change has been verified, the risk is low.
>> 
>> ![image](https://github.com/openjdk/jdk11u-dev/assets/24123821/63c82beb-1c90-45b0-b8b7-6419cf111252)
>> 
>> Thanks,
>> -sendao
>
> SendaoYan 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 two additional commits since the last revision:
> 
>  - Merge branch 'openjdk:master' into jbs8321925
>  - Backport b6233c3de773fb57b23704f1fec05d8b2d9c11c0
>    
>    Signed-off-by: sendaoYan <yansendao.ysd at alibaba-inc.com>

LGTM.

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

Marked as reviewed by mdoerr (Reviewer).

PR Review: https://git.openjdk.org/jdk11u-dev/pull/2631#pullrequestreview-2053740050