[IO-811] FileUtils.listFiles(File, String[], boolean) fails to close its

internal Stream

Different version of solution from PR #489 by Adam Rauch

Author: garydgregory

src/changes/changes.xml

@@ -55,7 +55,10 @@ The <action> type attribute can be add,update,fix,remove.
         Javadoc should mention closing Streams based on file resources.
       </action>
       <action dev="ggregory" type="fix" issue="IO-811" due-to="Adam Rauch, Gary Gregory">
-        In tests, Files.walk() direct and indirect callers fail to close the returned Stream&lt;Path&gt;.
+        In tests, Files.walk() direct and indirect callers fail to close the returned Stream.
+      </action>
+      <action dev="ggregory" type="fix" issue="IO-811" due-to="Adam Rauch, Gary Gregory">
+        FileUtils.listFiles(File, String[], boolean) fails to close its internal Stream.
       </action>
     </release>
     <release version="2.14.0" date="2023-09-24" description="Java 8 is required.">

src/main/java/org/apache/commons/io/FileUtils.java

@@ -2228,7 +2228,7 @@ public static Collection<File> listFiles(final File directory, final IOFileFilte
     }
 
     /**
-     * Finds files within a given directory (and optionally its subdirectories)
+     * Lists files within a given directory (and optionally its subdirectories)
      * which match an array of extensions.
      *
      * @param directory  the directory to search in
@@ -2238,7 +2238,9 @@ public static Collection<File> listFiles(final File directory, final IOFileFilte
      * @return a collection of java.io.File with the matching files
      */
     public static Collection<File> listFiles(final File directory, final String[] extensions, final boolean recursive) {
-        return Uncheck.apply(d -> toList(streamFiles(d, recursive, extensions)), directory);
+        try (Stream<File> fileStream = Uncheck.get(() -> streamFiles(directory, recursive, extensions))) {
+            return toList(fileStream);
+        }
     }
 
     /**