From f79dd096e2c85ef44f7b2d0f8e0c510ca73decb5 Mon Sep 17 00:00:00 2001
From: Looly <loolly@gmail.com>
Date: Fri, 31 Mar 2023 13:19:11 +0800
Subject: [PATCH] fix bug

---
 .../cn/hutool/core/io/file/PathMover.java     | 21 ++++++++++-----
 .../java/cn/hutool/core/io/FileUtilTest.java  | 26 +++++++++++++++++++
 2 files changed, 40 insertions(+), 7 deletions(-)

diff --git a/hutool-core/src/main/java/cn/hutool/core/io/file/PathMover.java b/hutool-core/src/main/java/cn/hutool/core/io/file/PathMover.java
index b6ae71158..8780d0359 100755
--- a/hutool-core/src/main/java/cn/hutool/core/io/file/PathMover.java
+++ b/hutool-core/src/main/java/cn/hutool/core/io/file/PathMover.java
@@ -15,6 +15,7 @@ package cn.hutool.core.io.file;
 import cn.hutool.core.io.IORuntimeException;
 import cn.hutool.core.io.file.visitor.MoveVisitor;
 import cn.hutool.core.lang.Assert;
+import cn.hutool.core.text.StrUtil;
 import cn.hutool.core.util.ObjUtil;
 
 import java.io.IOException;
@@ -82,27 +83,33 @@ public class PathMover {
 	 *     <li>如果src为文件，target为不存在的路径，则重命名源文件到目标指定的文件，如move("/a/b", "/c/d"), d不存在，则b变成d。</li>
 	 *     <li>如果src为目录，target为文件，抛出{@link IllegalArgumentException}</li>
 	 *     <li>如果src为目录，target为目录，则将源目录及其内容移动到目标路径目录中，如move("/a/b", "/c/d")，结果为"/c/d/b"</li>
+	 *     <li>如果src为目录，target为其子目录，抛出{@link IllegalArgumentException}</li>
 	 *     <li>如果src为目录，target为不存在的路径，则重命名src到target，如move("/a/b", "/c/d")，结果为"/c/d/"，相当于b重命名为d</li>
 	 * </ul>
 	 *
 	 * @return 目标文件Path
+	 * @throws IllegalArgumentException src为目录，target为其子目录抛出此异常
 	 */
-	public Path move() {
+	public Path move() throws IllegalArgumentException{
 		final Path src = this.src;
 		Path target = this.target;
 		final CopyOption[] options = this.options;
 
+		if (PathUtil.isSub(src, target)) {
+			if(Files.exists(target) && PathUtil.equals(src, target)){
+				// issue#2845，当用户传入目标路径与源路径一致时，直接返回，否则会导致删除风险。
+				return target;
+			}
+
+			// 当用户将文件夹拷贝到其子文件夹时，报错
+			throw new IllegalArgumentException(StrUtil.format("Target [{}] is sub path of src [{}]!", target, src));
+		}
+
 		if (PathUtil.isDirectory(target)) {
 			// 创建子路径的情况，1是目标是目录，需要移动到目录下，2是目标不能存在，自动创建目录
 			target = target.resolve(src.getFileName());
 		}
 
-		// issue#2893 target 不存在导致NoSuchFileException
-		if (Files.exists(target) && PathUtil.equals(src, target)) {
-			// issue#2845，当用户传入目标路径与源路径一致时，直接返回，否则会导致删除风险。
-			return target;
-		}
-
 		// 自动创建目标的父目录
 		PathUtil.mkParentDirs(target);
 		try {
diff --git a/hutool-core/src/test/java/cn/hutool/core/io/FileUtilTest.java b/hutool-core/src/test/java/cn/hutool/core/io/FileUtilTest.java
index f8418c82a..9cdde2503 100644
--- a/hutool-core/src/test/java/cn/hutool/core/io/FileUtilTest.java
+++ b/hutool-core/src/test/java/cn/hutool/core/io/FileUtilTest.java
@@ -69,6 +69,22 @@ public class FileUtilTest {
 		FileUtil.move(FileUtil.file("d:/test/a"), FileUtil.file("d:/test/b"), false);
 	}
 
+	@Test
+	@Disabled
+	public void renameToSubTest() {
+		Assertions.assertThrows(IllegalArgumentException.class, ()->{
+			// 移动到子目录，报错
+			FileUtil.move(FileUtil.file("d:/test/a"), FileUtil.file("d:/test/a/c"), false);
+		});
+	}
+
+	@Test
+	@Disabled
+	public void renameSameTest() {
+		// 目标和源相同，不处理
+		FileUtil.move(FileUtil.file("d:/test/a"), FileUtil.file("d:/test/a"), false);
+	}
+
 	@Test
 	public void copyTest() {
 		final File srcFile = FileUtil.file("hutool.jpg");
@@ -80,6 +96,16 @@ public class FileUtilTest {
 		Assertions.assertEquals(srcFile.length(), destFile.length());
 	}
 
+	@Test
+	@Disabled
+	public void copySameTest() {
+		final File srcFile = FileUtil.file("d:/test/a");
+		final File destFile = FileUtil.file("d:/test/");
+
+		// 拷贝到当前目录，不做处理
+		FileUtil.copy(srcFile, destFile, true);
+	}
+
 	@Test
 	@Disabled
 	public void copyDirTest() {