Fix apache#14923

oxsean · Nov 25, 2024 · 38cf318 · 38cf318
1 parent ac679e2
commit 38cf318
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/...in/java/org/apache/dubbo/rpc/protocol/tri/rest/mapping/DefaultRequestMappingRegistry.java b/...in/java/org/apache/dubbo/rpc/protocol/tri/rest/mapping/DefaultRequestMappingRegistry.java
@@ -180,6 +180,7 @@ public void unregister(Invoker<?> invoker) {
         if (tree == null) {
             return;
         }
+
         lock.writeLock().lock();
         try {
             tree.remove(mapping -> mapping.meta.getInvoker() == invoker);
@@ -193,6 +194,7 @@ public void destroy() {
         if (tree == null) {
             return;
         }
+
         lock.writeLock().lock();
         try {
             tree.clear();
@@ -202,6 +204,10 @@ public void destroy() {
     }
 
     public HandlerMeta lookup(HttpRequest request) {
+        if (tree == null) {
+            return null;
+        }
+
         String stringPath = PathUtils.normalize(request.uri());
         request.setAttribute(RestConstants.PATH_ATTRIBUTE, stringPath);
         KeyString path = new KeyString(stringPath, restConfig.getCaseSensitiveMatchOrDefault());
@@ -365,6 +371,10 @@ private void handleNoMatch(HttpRequest request, List<RequestMapping> partialMatc
 
     @Override
     public boolean exists(String stringPath, String method) {
+        if (tree == null) {
+            return false;
+        }
+
         KeyString path = new KeyString(stringPath, restConfig.getCaseSensitiveMatchOrDefault());
         if (tryExists(path, method)) {
             return true;