RFR: 8317128: java/nio/file/Files/CopyAndMove.java failed with AccessDeniedException [v2]

Brian Burkhalter bpb at openjdk.org
Mon Oct 2 18:51:02 UTC 2023


> The change #15501 removed explicit throwing of a `FileAlreadyExistsException` in `copyToForeignTarget` for non-directories when the target exists and `REPLACE_EXISTING` is not specified, instead relying on `FileChannel.open` eventually to throw this exception. The test, however, reuses the same file path, and on Windows `CreateFile`, which is invoked by `open`, can throw an `AccessDeniedException` if the file exists and has been marked for deletion but not yet actually deleted. This change proposes to reinstate explicitly throwing a FAEE.

Brian Burkhalter has updated the pull request incrementally with one additional commit since the last revision:

  8317128: Remove replaceExistingOrEmpty method

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/15991/files
  - new: https://git.openjdk.org/jdk/pull/15991/files/365149c7..4a34ff3b

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=15991&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=15991&range=00-01

  Stats: 9 lines in 1 file changed: 0 ins; 8 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/15991.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/15991/head:pull/15991

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


More information about the nio-dev mailing list