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

duke duke at openjdk.org
Tue Aug 13 22:54:56 UTC 2024


On Tue, 13 Aug 2024 20:24:21 GMT, Todd V. Jonker <duke at openjdk.org> wrote:

>> 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

@toddjonker 
Your change (at version a3bb19e58bb6ba7ef755ec86db21e903b594915a) is now ready to be sponsored by a Committer.

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

PR Comment: https://git.openjdk.org/jdk21u-dev/pull/863#issuecomment-2287316520


More information about the jdk-updates-dev mailing list