#890 Replace Permission#checkGuard with SecurityManager#checkPermission

Fixes `Connection.abort`, `Connection.setNetworkTimeout`, and `OperationMonitor.initOperationAware` always throwing _"java.lang.SecurityException: checking permissions is not supported"_ on Java 24.

Author: mrotteveel

src/docs/asciidoc/release_notes.adoc

@@ -44,6 +44,7 @@ This could also hang the cleaner thread used for closing leaked statements.
 * Fixed: `ResultSet` move incorrectly closes input `Clob` (https://github.com/FirebirdSQL/jaybird/issues/880[#880])
 * Fixed: Batch execution with multiple empty strings resulted in error _"Repeated blob id 0:0 in registerBlob()"_ (https://github.com/FirebirdSQL/jaybird/issues/888[#888])
 * Dependency update: updated `org.bouncycastle:bcprov-jdk18on` from 1.80 to 1.81 (used by `chacha64-plugin`) (https://github.com/FirebirdSQL/jaybird/issues/889[#889])
+* Fixed: On Java 24, `Connection.abort`, `Connection.setNetworkTimeout`, and `OperationMonitor.initOperationAware` always throw _"java.lang.SecurityException: checking permissions is not supported"_ (https://github.com/FirebirdSQL/jaybird/issues/890[#890])
 
 === Jaybird 6.0.2
 

src/main/org/firebirdsql/gds/ng/OperationMonitor.java

@@ -81,8 +81,12 @@ static void endOperation(Operation operation) {
      *         If a security manager is installed and the calling code does not have permission {@code
      *         "org.firebirdsql.jaybird.initOperationAware"}
      */
+    @SuppressWarnings("removal")
     public static void initOperationAware(OperationAware operationAware) {
-        PERMISSION_INIT_OPERATION_AWARE.checkGuard(OperationMonitor.class);
+        var sm = System.getSecurityManager();
+        if (sm != null) {
+            sm.checkPermission(PERMISSION_INIT_OPERATION_AWARE);
+        }
         instance.set(operationAware != null
                 ? operationAware
                 : NoOpOperationAware.INSTANCE);

src/main/org/firebirdsql/jdbc/FBConnection.java

@@ -1282,11 +1282,14 @@ public void resetKnownClientInfoProperties() {
     }
 
     @Override
-    @SuppressWarnings("java:S1141")
+    @SuppressWarnings({ "java:S1141", "removal" })
     public void abort(Executor executor) throws SQLException {
         // NOTE: None of these operations are performed under lock (except maybe very late in the cleanup)
         if (isClosed()) return;
-        PERMISSION_CALL_ABORT.checkGuard(this);
+        var sm = System.getSecurityManager();
+        if (sm != null) {
+            sm.checkPermission(PERMISSION_CALL_ABORT);
+        }
         if (executor == null) {
             throw FbExceptionBuilder.toException(jb_invalidExecutor);
         }
@@ -1328,9 +1331,13 @@ public void abort(Executor executor) throws SQLException {
         });
     }
 
+    @SuppressWarnings("removal")
     @Override
     public void setNetworkTimeout(Executor executor, int milliseconds) throws SQLException {
-        PERMISSION_SET_NETWORK_TIMEOUT.checkGuard(this);
+        var sm = System.getSecurityManager();
+        if (sm != null) {
+            sm.checkPermission(PERMISSION_SET_NETWORK_TIMEOUT);
+        }
         if (executor == null) {
             throw FbExceptionBuilder.toException(jb_invalidExecutor);
         }