8307347: serviceability/sa/ClhsdbDumpclass.java could leave files own…

…ed by root on macOS Backport-of: 5c7ede94ae59b46c12d40a38bf5b7e15319cc7e2
openjdk · May 25, 2023 · 50aabaa · 50aabaa · openjdk-notifier · May 25, 2023
1 parent 861fe74
commit 50aabaa
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/test/hotspot/jtreg/serviceability/sa/ClhsdbDumpclass.java b/test/hotspot/jtreg/serviceability/sa/ClhsdbDumpclass.java
@@ -31,6 +31,7 @@
 import jdk.test.lib.apps.LingeredApp;
 import jdk.test.lib.JDKToolLauncher;
 import jdk.test.lib.process.OutputAnalyzer;
+import jdk.test.lib.SA.SATestUtils;
 import jtreg.SkippedException;
 
 /**
@@ -47,6 +48,10 @@ public class ClhsdbDumpclass {
     static final String APP_SLASH_CLASSNAME = APP_DOT_CLASSNAME.replace('.', '/');
 
     public static void main(String[] args) throws Exception {
+        if (SATestUtils.needsPrivileges()) {
+            // This test will create a file as root that cannot be easily deleted, so don't run it.
+            throw new SkippedException("Cannot run this test on OSX if adding privileges is required.");
+        }
         System.out.println("Starting ClhsdbDumpclass test");
 
         LingeredApp theApp = null;