Fix Urls may be null, and NullPointerException will be thrown in Conf…

…igValidationUtils (apache#8020) Signed-off-by: xingrufei <xingrufei@sogou-inc.com>
skyguard1 · Jun 9, 2021 · 9bbc8db · 9bbc8db
1 parent 584ab28
commit 9bbc8db
Showing 1 changed file with 11 additions and 9 deletions.
diff --git a/...g/dubbo-config-api/src/main/java/org/apache/dubbo/config/utils/ConfigValidationUtils.java b/...g/dubbo-config-api/src/main/java/org/apache/dubbo/config/utils/ConfigValidationUtils.java
@@ -205,15 +205,17 @@ public static List<URL> loadRegistries(AbstractInterfaceConfig interfaceConfig,
                     }
                     List<URL> urls = UrlUtils.parseURLs(address, map);
 
-                    for (URL url : urls) {
-
-                        url = URLBuilder.from(url)
-                                .addParameter(REGISTRY_KEY, url.getProtocol())
-                                .setProtocol(extractRegistryType(url))
-                                .build();
-                        if ((provider && url.getParameter(REGISTER_KEY, true))
-                                || (!provider && url.getParameter(SUBSCRIBE_KEY, true))) {
-                            registryList.add(url);
+                    if (CollectionUtils.isNotEmpty(urls)) {
+                        for (URL url : urls) {
+
+                            url = URLBuilder.from(url)
+                                    .addParameter(REGISTRY_KEY, url.getProtocol())
+                                    .setProtocol(extractRegistryType(url))
+                                    .build();
+                            if ((provider && url.getParameter(REGISTER_KEY, true))
+                                    || (!provider && url.getParameter(SUBSCRIBE_KEY, true))) {
+                                registryList.add(url);
+                            }
                         }
                     }
                 }