Skip to content

Commit 0ca426e

Browse files
committed
Return errc::no_such_file_or_directory in fs::access if GetFileAttributesW fails (llvm#83495)
Fixes llvm#83046 There is a race condition when calling `GetFileAttributesW` that can cause it to return `ERROR_ACCESS_DENIED` on a path which exists, which is unexpected for callers using this function to check for file existence by passing `AccessMode::Exist`. This was manifesting as a compiler crash on Windows downstream in the Swift compiler when using the `-index-store-path` flag (more information in #8224). I looked for alternate APIs to avoid bringing in `shlwapi.h`, but didn't see any good candidates. I'm not tied at all to this solution, any feedback and alternative approaches are more than welcome. (cherry picked from commit 9961c03)
1 parent fce7267 commit 0ca426e

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

llvm/lib/Support/Windows/Path.inc

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -630,6 +630,10 @@ std::error_code access(const Twine &Path, AccessMode Mode) {
630630
DWORD Attributes = ::GetFileAttributesW(PathUtf16.begin());
631631

632632
if (Attributes == INVALID_FILE_ATTRIBUTES) {
633+
// Avoid returning unexpected error codes when querying for existence.
634+
if (Mode == AccessMode::Exist)
635+
return errc::no_such_file_or_directory;
636+
633637
// See if the file didn't actually exist.
634638
DWORD LastError = ::GetLastError();
635639
if (LastError != ERROR_FILE_NOT_FOUND && LastError != ERROR_PATH_NOT_FOUND)

0 commit comments

Comments
 (0)