Stop using ValidationSecurityManager as SecurityManager

The Security Manager is deprecated and subject to removal in a future release. There is no replacement for the Security Manager.

See also spotbugs#1579

CHANGELOG.md

@@ -7,6 +7,7 @@ Currently the versioning policy of this project follows [Semantic Versioning v2.
 ## Unreleased - 2022-??-??
 ### Fixed
 - Bumped Saxon-HE from 10.6 to 11.2 ([#1955](https://github.com/spotbugs/spotbugs/pull/1955))
+- Warnings of deprecated System::setSecurityManager calls on Java 17
 
 ## 4.6.0 - 2022-03-08
 ### Fixed

spotbugs/src/main/java/edu/umd/cs/findbugs/ba/jsr305/TypeQualifierValue.java

@@ -148,15 +148,6 @@ private TypeQualifierValue(ClassDescriptor typeQualifier, @CheckForNull Object v
                 try {
                     Global.getAnalysisCache().getClassAnalysis(ClassData.class, checkerName);
 
-                    // found it.
-                    SecurityManager m = System.getSecurityManager();
-                    if (m == null) {
-                        if (DEBUG_CLASSLOADING) {
-                            System.out.println("Setting ValidationSecurityManager");
-                        }
-                        System.setSecurityManager(ValidationSecurityManager.INSTANCE);
-                    }
-
                     Class<?> c = ValidatorClassLoader.INSTANCE.loadClass(checkerName.getDottedClassName());
                     if (TypeQualifierValidator.class.isAssignableFrom(c)) {
 
@@ -187,14 +178,6 @@ private TypeQualifierValue(ClassDescriptor typeQualifier, @CheckForNull Object v
                     AnalysisContext.logError("Unable to construct type qualifier checker " + checkerName + " due to "
                             + e.getClass().getSimpleName() + ":" + e.getMessage());
                 }
-            } else if (DEBUG_CLASSLOADING) {
-                SecurityManager m = System.getSecurityManager();
-                if (m == null) {
-                    if (DEBUG_CLASSLOADING) {
-                        System.out.println("Setting ValidationSecurityManager");
-                    }
-                    System.setSecurityManager(ValidationSecurityManager.INSTANCE);
-                }
             }
         }
         this.validator = validator1;
@@ -269,7 +252,7 @@ public When validate(@CheckForNull Object constantValue) {
         Profiler profiler = analysisCache.getProfiler();
         profiler.start(validator.getClass());
         try {
-            return ValidationSecurityManager.sandboxedValidation(proxy, validator, constantValue);
+            return validator.forConstantValue(proxy, constantValue);
         } catch (Exception e) {
             AnalysisContext.logError("Error executing custom validator for " + typeQualifier + " " + constantValue, e);
             return When.UNKNOWN;