(iotdb) 01/01: Fix deleteDirectoryAndEmptyParent NPE

[haonan]

This is an automated email from the ASF dual-hosted git repository.

haonan pushed a commit to branch fix_delete_file_npe
in repository https://gitbox.apache.org/repos/asf/iotdb.git

commit e73ba036348dd18f96160dabc50e1404735430c6
Author: HTHou <hhao...@outlook.com>
AuthorDate: Fri Nov 22 16:58:50 2024 +0800

    Fix deleteDirectoryAndEmptyParent NPE
---
 .../src/main/java/org/apache/iotdb/commons/utils/FileUtils.java      | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git 
a/iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/utils/FileUtils.java
 
b/iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/utils/FileUtils.java
index b4edc5164a2..99a7b4e7589 100644
--- 
a/iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/utils/FileUtils.java
+++ 
b/iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/utils/FileUtils.java
@@ -42,7 +42,6 @@ import java.nio.file.StandardCopyOption;
 import java.text.CharacterIterator;
 import java.text.StringCharacterIterator;
 import java.util.Arrays;
-import java.util.Objects;
 
 public class FileUtils {
   private static final Logger LOGGER = 
LoggerFactory.getLogger(FileUtils.class);
@@ -90,8 +89,8 @@ public class FileUtils {
   public static void deleteDirectoryAndEmptyParent(File folder) {
     deleteFileOrDirectory(folder);
     final File parentFolder = folder.getParentFile();
-    if (parentFolder.isDirectory()
-        && Objects.requireNonNull(parentFolder.listFiles()).length == 0) {
+    File[] files = parentFolder.listFiles();
+    if (parentFolder.isDirectory() && (files == null || files.length == 0)) {
       if (!parentFolder.delete()) {
         LOGGER.warn("Delete folder failed: {}", 
parentFolder.getAbsolutePath());
       }