PathUtils.deleteOnExit(Path) calls Objects.requireNonNull() on the wrong

object
wqou · May 3, 2024 · 0989442 · 0989442
1 parent 4b10ee6
commit 0989442
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/changes/changes.xml b/src/changes/changes.xml
@@ -50,6 +50,7 @@ The <action> type attribute can be add,update,fix,remove.
       <!-- FIX -->
       <action dev="ggregory" type="fix"                due-to="Dependabot">Add missing unit tests.</action>
       <action dev="ggregory" type="fix"                due-to="Dependabot">FileUtils.lastModifiedFileTime(File) calls Objects.requireNonNull() on the wrong object.</action>
+      <action dev="ggregory" type="fix"                due-to="Dependabot">PathUtils.deleteOnExit(Path) calls Objects.requireNonNull() on the wrong object.</action>
       <!-- UPDATE -->
       <action dev="ggregory" type="update"             due-to="Dependabot">Bump tests commons.bytebuddy.version from 1.14.13 to 1.14.14 #615.</action>
       <action dev="ggregory" type="update"             due-to="Dependabot">Bump tests commons-codec:commons-codec from 1.16.1 to 1.17.0.</action>

diff --git a/src/main/java/org/apache/commons/io/file/PathUtils.java b/src/main/java/org/apache/commons/io/file/PathUtils.java
@@ -605,7 +605,7 @@ public static PathCounters deleteFile(final Path file, final LinkOption[] linkOp
      * @since 3.13.0
      */
     public static void deleteOnExit(final Path path) {
-        Objects.requireNonNull(path.toFile()).deleteOnExit();
+        Objects.requireNonNull(path).toFile().deleteOnExit();
     }
 
     /**