jvmSandbox对象在一定情况下可能为空，导致这种情况的可能是destroy()调用发生在bind()方法调用之前

所以这里做了一个判空处理，临时性解决这个问题。真正需要深究的是为什么destroy()竟然能在bind()之前被调用
darcydai · Dec 11, 2019 · be731d7 · be731d7
1 parent 6fe5e48
commit be731d7
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/sandbox-core/src/main/java/com/alibaba/jvm/sandbox/core/server/jetty/JettyCoreServer.java b/sandbox-core/src/main/java/com/alibaba/jvm/sandbox/core/server/jetty/JettyCoreServer.java
@@ -81,7 +81,7 @@ public void process() throws Throwable {
 
             // destroy http server
             logger.info("{} is destroying", this);
-            while (!httpServer.isStopped());
+            while (!httpServer.isStopped()) ;
             httpServer.destroy();
 
         } catch (Throwable cause) {
@@ -221,7 +221,14 @@ public void process() throws Throwable {
     public void destroy() {
 
         // 关闭JVM-SANDBOX
-        jvmSandbox.destroy();
+        /*
+         * BUGFIX:
+         * jvmSandbox对象在一定情况下可能为空，导致这种情况的可能是destroy()调用发生在bind()方法调用之前
+         * 所以这里做了一个判空处理，临时性解决这个问题。真正需要深究的是为什么destroy()竟然能在bind()之前被调用
+         */
+        if (null != jvmSandbox) {
+            jvmSandbox.destroy();
+        }
 
         // 关闭HTTP服务器
         if (isBind()) {