spotbugs

Wetator · Apr 27, 2022 · 9de6521 · 9de6521
1 parent 3aea495
commit 9de6521
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/src/org/wetator/backend/htmlunit/ResponseStore.java b/src/org/wetator/backend/htmlunit/ResponseStore.java
@@ -126,12 +126,17 @@ private void initOutputDir(final String aBrowserSubdir) {
   }
 
   private static void cleanDirectory(final File aDirectory) {
-    for (File tmpFile : aDirectory.listFiles()) {
-      if (tmpFile.isDirectory()) {
-        cleanDirectory(tmpFile);
-      }
+    final File[] tmpFiles = aDirectory.listFiles();
+    if (tmpFiles != null) {
+      for (File tmpFile : tmpFiles) {
+        if (tmpFile.isDirectory()) {
+          cleanDirectory(tmpFile);
+        }
 
-      tmpFile.delete();
+        if (!tmpFile.delete()) {
+          LOG.error("Can't delete file '" + FilenameUtils.normalize(tmpFile.getAbsolutePath()) + "'");
+        }
+      }
     }
   }