[jdk21u-dev] RFR: 8073061: (fs) Files.copy(foo, bar, REPLACE_EXISTING) deletes bar even if foo is not readable [v2]

Todd V. Jonker duke at openjdk.org
Tue Aug 13 20:24:21 UTC 2024


> This is a clean backport of https://github.com/openjdk/jdk/commit/36ac83904c9e81a01822b0e36ef677cae2808709 originally authored by @bplb and reviewed by @AlanBateman  
> 
> It also includes this follow-up fix:
>   * [JDK-8317128](https://bugs.openjdk.org/browse/JDK-8317128): **java/nio/file/Files/CopyAndMove.java failed with AccessDeniedException** 
>       * https://github.com/openjdk/jdk/pull/15991
> 
> Additional testing:
> - [x] New regression test fails without the fix, passes with it
> - [x] linux-x86_64-server-release: `JTREG_KEYWORDS="!headful & !external-dep & !printer" make test TEST="tier1 tier2 tier3"`

Todd V. Jonker has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:

 - Merge branch 'master' into filedelete for testing
 - Clean JDK-8317128: java/nio/file/Files/CopyAndMove.java failed with AccessDeniedException
   
   6c6beba2569a2f9f3fd5d6988360ffd8680de821
 - Backport 36ac83904c9e81a01822b0e36ef677cae2808709

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

Changes:
  - all: https://git.openjdk.org/jdk21u-dev/pull/863/files
  - new: https://git.openjdk.org/jdk21u-dev/pull/863/files/a650c6ef..a3bb19e5

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk21u-dev&pr=863&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk21u-dev&pr=863&range=00-01

  Stats: 5796 lines in 130 files changed: 2630 ins; 1925 del; 1241 mod
  Patch: https://git.openjdk.org/jdk21u-dev/pull/863.diff
  Fetch: git fetch https://git.openjdk.org/jdk21u-dev.git pull/863/head:pull/863

PR: https://git.openjdk.org/jdk21u-dev/pull/863


More information about the jdk-updates-dev mailing list