From 659aca0a65490d08dd2fbb33ebbe591cc2058367 Mon Sep 17 00:00:00 2001
From: justparking <justparking@gmail.com>
Date: Tue, 11 Apr 2017 14:58:31 +1000
Subject: [PATCH] minor: NodelClients.list() should return empty array on
 lookup miss instead of throwing NullPointer exception.

---
 .../src/main/java/org/nodel/core/NodelClients.java        | 8 +++++---
 1 file changed, 5 insertions(+), 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
index 0dd3da4c..5b14da8e 100644
--- 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>();