Skip to content

Commit 7341439

Browse files
committed
8279007: jstatd fails to start because SecurityManager is disabled
Reviewed-by: alanb, sspitsyn, mullan
1 parent 97c5cd7 commit 7341439

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

make/modules/jdk.jstatd/Launcher.gmk

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
#
2-
# Copyright (c) 2011, 2020, Oracle and/or its affiliates. All rights reserved.
2+
# Copyright (c) 2011, 2021, Oracle and/or its affiliates. All rights reserved.
33
# DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
44
#
55
# This code is free software; you can redistribute it and/or modify it
@@ -27,4 +27,5 @@ include LauncherCommon.gmk
2727

2828
$(eval $(call SetupBuildLauncher, jstatd, \
2929
MAIN_CLASS := sun.tools.jstatd.Jstatd, \
30+
JAVA_ARGS := -Djava.security.manager=allow, \
3031
))

test/jdk/sun/tools/jstatd/JstatdTest.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -253,7 +253,6 @@ private void cleanUpThread(ProcessThread thread) throws Throwable {
253253
private String[] getJstatdCmd() throws Exception {
254254
JDKToolLauncher launcher = JDKToolLauncher.createUsingTestJDK("jstatd");
255255
launcher.addVMArg("-XX:+UsePerfData");
256-
launcher.addVMArg("-Djava.security.manager=allow");
257256
String testSrc = System.getProperty("test.src");
258257
File policy = new File(testSrc, "all.policy");
259258
assertTrue(policy.exists() && policy.isFile(),

0 commit comments

Comments
 (0)