chore: close stream to prevent possible resource leak (#6382)

jeremylong · Jan 13, 2024 · a6c3909 · a6c3909
1 parent 18c70f7
commit a6c3909
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/utils/src/main/java/org/owasp/dependencycheck/utils/FileUtils.java b/utils/src/main/java/org/owasp/dependencycheck/utils/FileUtils.java
@@ -28,6 +28,7 @@
 import java.nio.file.Path;
 import java.util.Comparator;
 import java.util.UUID;
+import java.util.stream.Stream;
 
 import org.apache.commons.io.FilenameUtils;
 import org.apache.commons.lang3.StringUtils;
@@ -90,9 +91,8 @@ public static boolean delete(@Nullable File file) {
  return false;
  }
 
- try {
- Files.walk(file.toPath())
- .sorted(Comparator.reverseOrder())
+ try (Stream<Path> paths = Files.walk(file.toPath())) {
+ paths.sorted(Comparator.reverseOrder())
  .map(Path::toFile)
  .forEach(File::delete);
  } catch (IOException ex) {