On Tue, 9 Jun 2026 19:11:21 GMT, Shiv Shah <[email protected]> wrote:

> FileUtils.copyDirectory() used Files.copy() for all entries including 
> directories. When the destination is an existing directory (like PWD), 
> Files.copy with REPLACE_EXISTING replaces the directory inode, corrupting it. 
> Fixed by using Files.createDirectories() for directory entries and 
> Files.copy() only for files. Added test verifying copyDirectory to PWD works 
> correctly.
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

This pull request has now been integrated.

Changeset: 911e50be
Author:    Shiv Shah <[email protected]>
Committer: Leonid Mesnik <[email protected]>
URL:       
https://git.openjdk.org/jdk/commit/911e50be4c9172b3315c88e3348a76ab1cf58f41
Stats:     36 lines in 2 files changed: 32 ins; 0 del; 4 mod

8355320: FileUtils.copyDirectory() mishandles an existing destination directory

Reviewed-by: lmesnik

-------------

PR: https://git.openjdk.org/jdk/pull/31444

Reply via email to