minor: NodelClients.list() should return empty array on lookup miss i…

…nstead of throwing NullPointer exception.
museumsvictoria · Apr 11, 2017 · 659aca0 · 659aca0
1 parent 22f97a1
commit 659aca0
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/nodel-framework-java/src/main/java/org/nodel/core/NodelClients.java b/nodel-framework-java/src/main/java/org/nodel/core/NodelClients.java
@@ -888,10 +888,12 @@ protected List<NodeURL> getAllNodesURLs() throws IOException {
      */
     protected List<NodeURL> getNodeURLs(SimpleName name) throws IOException {
         Collection<AdvertisementInfo> list;
-        if (name == null)
+        if (name == null) {
             list = AutoDNS.instance().list();
-        else
-            list = Arrays.asList(AutoDNS.instance().resolve(name));
+        } else {
+            AdvertisementInfo result = AutoDNS.instance().resolve(name);
+            list = result != null ? Arrays.asList(result) : Collections.<AdvertisementInfo>emptyList();
+        }
 
         List<NodeURL> nodeURLs = new ArrayList<NodeURL>();