[7.6.0] Don't attempt to enable the Security Manager for jdk >= 24 (#25658)

hvadehra · web-flow · commit 806a6e823209 · 2025-03-21T10:38:27.000Z
Backports bazelbuild/rules_java@b3a0580 Fixes #25648
diff --git a/src/main/starlark/builtins_bzl/bazel/java/bazel_java_binary.bzl b/src/main/starlark/builtins_bzl/bazel/java/bazel_java_binary.bzl
@@ -64,7 +64,8 @@ def _bazel_base_binary_impl(ctx, is_test_rule_class):
     )
 
     if ctx.attr.use_testrunner:
-        if semantics.find_java_runtime_toolchain(ctx).version >= 17:
+        _java_runtime_version = semantics.find_java_runtime_toolchain(ctx).version
+        if _java_runtime_version >= 17 and _java_runtime_version < 24:
             jvm_flags.append("-Djava.security.manager=allow")
         test_class = ctx.attr.test_class if hasattr(ctx.attr, "test_class") else ""
         if test_class == "":

Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,8 @@ def _bazel_base_binary_impl(ctx, is_test_rule_class):`
`64`	`64`	`)`
`65`	`65`
`66`	`66`	`if ctx.attr.use_testrunner:`
`67`		`- if semantics.find_java_runtime_toolchain(ctx).version >= 17:`
	`67`	`+ _java_runtime_version = semantics.find_java_runtime_toolchain(ctx).version`
	`68`	`+ if _java_runtime_version >= 17 and _java_runtime_version < 24:`
`68`	`69`	`jvm_flags.append("-Djava.security.manager=allow")`
`69`	`70`	`test_class = ctx.attr.test_class if hasattr(ctx.attr, "test_class") else ""`
`70`	`71`	`if test_class == "":`