From 6698a314eec125f675cf18432237744eb7b9a69f Mon Sep 17 00:00:00 2001
From: Greg Wilkins <gregw@webtide.com>
Date: Wed, 28 Oct 2020 15:23:29 +0100
Subject: [PATCH] Fixes #5521 ResourceCollection list NPE (#5523)

Fixes #5521 ResourceCollection list NPE
---
 .../jetty/util/resource/PathResource.java       |  6 +-----
 .../jetty/util/resource/ResourceCollection.java |  5 +++--
 .../util/resource/ResourceCollectionTest.java   | 17 +++++++++++++++++
 3 files changed, 21 insertions(+), 7 deletions(-)

diff --git a/jetty-util/src/main/java/org/eclipse/jetty/util/resource/PathResource.java b/jetty-util/src/main/java/org/eclipse/jetty/util/resource/PathResource.java
index e9931f548e43..e29e285c4b11 100644
--- a/jetty-util/src/main/java/org/eclipse/jetty/util/resource/PathResource.java
+++ b/jetty-util/src/main/java/org/eclipse/jetty/util/resource/PathResource.java
@@ -571,11 +571,7 @@ public String[] list()
             int size = entries.size();
             return entries.toArray(new String[size]);
         }
-        catch (DirectoryIteratorException e)
-        {
-            LOG.debug(e);
-        }
-        catch (IOException e)
+        catch (DirectoryIteratorException | IOException e)
         {
             LOG.debug(e);
         }
diff --git a/jetty-util/src/main/java/org/eclipse/jetty/util/resource/ResourceCollection.java b/jetty-util/src/main/java/org/eclipse/jetty/util/resource/ResourceCollection.java
index 594ba49c3b3c..3f080072465a 100644
--- a/jetty-util/src/main/java/org/eclipse/jetty/util/resource/ResourceCollection.java
+++ b/jetty-util/src/main/java/org/eclipse/jetty/util/resource/ResourceCollection.java
@@ -434,11 +434,12 @@ public long length()
     public String[] list()
     {
         assertResourcesSet();
-
         HashSet<String> set = new HashSet<>();
         for (Resource r : _resources)
         {
-            Collections.addAll(set, r.list());
+            String[] list = r.list();
+            if (list != null)
+                Collections.addAll(set, list);
         }
         String[] result = set.toArray(new String[0]);
         Arrays.sort(result);
diff --git a/jetty-util/src/test/java/org/eclipse/jetty/util/resource/ResourceCollectionTest.java b/jetty-util/src/test/java/org/eclipse/jetty/util/resource/ResourceCollectionTest.java
index 51200037cce0..d3e212517c6f 100644
--- a/jetty-util/src/test/java/org/eclipse/jetty/util/resource/ResourceCollectionTest.java
+++ b/jetty-util/src/test/java/org/eclipse/jetty/util/resource/ResourceCollectionTest.java
@@ -22,6 +22,7 @@
 import java.io.File;
 import java.io.InputStreamReader;
 import java.nio.file.Path;
+import java.util.Arrays;
 
 import org.eclipse.jetty.toolchain.test.FS;
 import org.eclipse.jetty.toolchain.test.MavenTestingUtils;
@@ -32,7 +33,9 @@
 import org.junit.jupiter.api.extension.ExtendWith;
 
 import static org.hamcrest.MatcherAssert.assertThat;
+import static org.hamcrest.Matchers.contains;
 import static org.hamcrest.Matchers.is;
+import static org.hamcrest.Matchers.nullValue;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertThrows;
 import static org.junit.jupiter.api.Assertions.assertTrue;
@@ -174,6 +177,20 @@ private void assertThrowIllegalStateException(ResourceCollection coll)
         });
     }
 
+    @Test
+    public void testList() throws Exception
+    {
+        ResourceCollection rc1 = new ResourceCollection(
+            Resource.newResource("src/test/resources/org/eclipse/jetty/util/resource/one/"),
+            Resource.newResource("src/test/resources/org/eclipse/jetty/util/resource/two/"),
+            Resource.newResource("src/test/resources/org/eclipse/jetty/util/resource/three/"));
+
+        assertThat(Arrays.asList(rc1.list()), contains("1.txt", "2.txt", "3.txt", "dir/"));
+        assertThat(Arrays.asList(rc1.addPath("dir").list()), contains("1.txt", "2.txt", "3.txt"));
+        assertThat(rc1.addPath("unknown").list(), nullValue());
+        // TODO for jetty-10 assertThat(rc1.addPath("unknown").list(), nullValue());
+    }
+
     @Test
     public void testMultipleSources1() throws Exception
     {