Reduce overhead of the security manager

nailgun-server/src/main/java/com/martiansoftware/nailgun/NGSecurityManager.java

@@ -18,6 +18,8 @@
 
 package com.martiansoftware.nailgun;
 
+import java.io.FileDescriptor;
+import java.net.InetAddress;
 import java.security.Permission;
 
 /**
@@ -58,10 +60,144 @@ public void checkPermission(Permission perm, Object context) {
     }
   }
 
-  /** Avoid constructing a FilePermission object in checkRead if base manager is null. */
+  // Overrides below avoid the cost of creating Permissions objects if base manager is null.
+  // FilePermission, in particular, is expensive to create. 
+
   public void checkRead(String file) {
     if (base != null) {
-      super.checkRead(file);
+      base.checkRead(file);
+    }
+  }
+
+  public void checkCreateClassLoader() {
+    if (base != null) {
+      base.checkCreateClassLoader();
+    }
+  }
+
+  public void checkAccess(Thread t) {
+    if (base != null) {
+      base.checkAccess(t);
+    }
+  }
+
+  public void checkAccess(ThreadGroup g) {
+    if (base != null) {
+      base.checkAccess(g);
+    }
+  }
+
+  public void checkExec(String cmd) {
+    if (base != null) {
+      base.checkExec(cmd);
+    }
+  }
+
+  public void checkLink(String lib) {
+    if (base != null) {
+      base.checkLink(lib);
+    }
+  }
+
+  public void checkRead(FileDescriptor fd) {
+    if (base != null) {
+      base.checkRead(fd);
+    }
+  }
+
+  public void checkRead(String file, Object context) {
+    if (base != null) {
+      base.checkRead(file, context);
+    }
+  }
+
+  public void checkWrite(FileDescriptor fd) {
+    if (base != null) {
+      base.checkWrite(fd);
+    }
+  }
+
+  public void checkWrite(String file) {
+    if (base != null) {
+      base.checkWrite(file);
+    }
+  }
+
+  public void checkDelete(String file) {
+    if (base != null) {
+      base.checkDelete(file);
+    }
+  }
+
+  public void checkConnect(String host, int port) {
+    if (base != null) {
+      base.checkConnect(host, port);
+    }
+  }
+
+  public void checkConnect(String host, int port, Object context) {
+    if (base != null) {
+      base.checkConnect(host, port, context);
+    }
+  }
+
+  public void checkListen(int port) {
+    if (base != null) {
+      base.checkListen(port);
+    }
+  }
+
+  public void checkAccept(String host, int port) {
+    if (base != null) {
+      base.checkAccept(host, port);
+    }
+  }
+
+  public void checkMulticast(InetAddress maddr) {
+    if (base != null) {
+      base.checkMulticast(maddr);
+    }
+  }
+
+  public void checkPropertiesAccess() {
+    if (base != null) {
+      base.checkPropertiesAccess();
+    }
+  }
+
+  public void checkPropertyAccess(String key) {
+    if (base != null) {
+      base.checkPropertyAccess(key);
+    }
+  }
+
+  public void checkPrintJobAccess() {
+    if (base != null) {
+      base.checkPrintJobAccess();
+    }
+  }
+
+  public void checkPackageAccess(String pkg) {
+    if (base != null) {
+      base.checkPackageAccess(pkg);
+    }
+  }
+
+  public void checkPackageDefinition(String pkg) {
+    if (base != null) {
+      base.checkPackageDefinition(pkg);
+    }
+  }
+
+  public void checkSetFactory() {
+    if (base != null) {
+      base.checkSetFactory();
+    }
+  }
+
+  public void checkSecurityAccess(String target) {
+    if (base != null) {
+      base.checkSecurityAccess(target);
     }
   }
 }