On Thu, 20 Nov 2025 13:05:44 GMT, Neha Joshi <[email protected]> wrote:
> In this PR we have updated the code to remove the duplicate call and ensure
> JVM flags are added only once.
>
> ### ISSUE :-
> In the test case -->
> ProcessTools.createTestJavaProcessBuilder(Utils.addTestJavaOpts("SSLEngineKeyLimit",
> "p", args[1], args[2]));
>
> -> Before executing ProcessTools.createTestJavaProcessBuilder() we are
> calling ---> Utils.addTestJavaOpts() which internally call
> prependTestJavaOpts();
>
>> public static String[] addTestJavaOpts(String... userArgs) {
>> return prependTestJavaOpts(userArgs);
>> }
>
> -> Then again inside ProcessTools.createTestJavaProcessBuilder() method we
> are calling Utils.prependTestJavaOpts()
>
>> public static ProcessBuilder createTestJavaProcessBuilder(String... command)
>> {
>> return createJavaProcessBuilder(Utils.prependTestJavaOpts(command));
>> }
>
>
> Calling Utils.prependTestJavaOpts() twice leads to duplicate JVM flags (This
> method call-> getTestJavaOpts() the main reason for duplication and below is
> the implementation for the same. )
>
>> public static String[] getTestJavaOpts() {
>> List<String> opts = new ArrayList<String>();
>> Collections.addAll(opts, safeSplitString(VM_OPTIONS));
>> Collections.addAll(opts, safeSplitString(JAVA_OPTIONS));
>> return opts.toArray(new String[0]);
>> }
>
> Contributed-by: @Korov
> https://github.com/openjdk/jdk/pull/26404
>
> https://bugs.openjdk.org/browse/JDK-8362658
This pull request has now been integrated.
Changeset: 520c092a
Author: Neha Joshi <[email protected]>
Committer: Rajan Halade <[email protected]>
URL:
https://git.openjdk.org/jdk/commit/520c092a658559a5d65f06a51061db3aae09931e
Stats: 36 lines in 8 files changed: 0 ins; 29 del; 7 mod
8362658: sun/security/ssl/SSLEngineImpl/* tests duplicate jvm flags
Co-authored-by: Lei Zhu <[email protected]>
Reviewed-by: myankelevich, rhalade
-------------
PR: https://git.openjdk.org/jdk/pull/28428
