diff options
author | Vladislav Vaintroub <wlad@mariadb.com> | 2021-05-03 15:34:52 +0200 |
---|---|---|
committer | Vladislav Vaintroub <wlad@mariadb.com> | 2021-05-03 15:35:00 +0200 |
commit | a910e1ee9e6185fbc2c29bca58f02616381d8a36 (patch) | |
tree | 2d4863b764383ee776871c9103416fc600d0ac00 /mysys | |
parent | abe6eb10a65d5c28c221d756357b4e7f392ad13d (diff) | |
download | mariadb-git-a910e1ee9e6185fbc2c29bca58f02616381d8a36.tar.gz |
MDEV-25584 Implement posix semantics file deletion for Windows 10
Diffstat (limited to 'mysys')
-rw-r--r-- | mysys/my_delete.c | 33 |
1 files changed, 27 insertions, 6 deletions
diff --git a/mysys/my_delete.c b/mysys/my_delete.c index 4201c252715..6cdecef22f1 100644 --- a/mysys/my_delete.c +++ b/mysys/my_delete.c @@ -58,6 +58,7 @@ int my_delete(const char *name, myf MyFlags) #if defined (_WIN32) + /* Delete file. @@ -65,15 +66,14 @@ int my_delete(const char *name, myf MyFlags) where another program (or thread in the current program) has the the same file open. - We're using 2 tricks to prevent the errors. + We're using several tricks to prevent the errors, such as + + - Windows 10 "posix semantics" delete - 1. A usual Win32's DeleteFile() can with ERROR_SHARED_VIOLATION, - because the file is opened in another application (often, antivirus or backup) - - We avoid the error by using CreateFile() with FILE_FLAG_DELETE_ON_CLOSE, instead + - Avoid the error by using CreateFile() with FILE_FLAG_DELETE_ON_CLOSE, instead of DeleteFile() - 2. If file which is deleted (delete on close) but has not entirely gone, + - If file which is deleted (delete on close) but has not entirely gone, because it is still opened by some app, an attempt to trcreate file with the same name would result in yet another error. The workaround here is renaming a file to unique name. @@ -116,6 +116,27 @@ static int my_win_unlink(const char *name) DBUG_RETURN(0); } + /* + Try Windows 10 method, delete with "posix semantics" (file is not visible, and creating + a file with the same name won't fail, even if it the fiile was open) + */ + struct + { + DWORD _Flags; + } disp={0x3}; + /* 0x3 = FILE_DISPOSITION_FLAG_DELETE | FILE_DISPOSITION_FLAG_POSIX_SEMANTICS */ + + handle= CreateFile(name, DELETE, FILE_SHARE_READ | FILE_SHARE_WRITE | FILE_SHARE_DELETE, + NULL, OPEN_EXISTING, 0, NULL); + if (handle != INVALID_HANDLE_VALUE) + { + BOOL ok= SetFileInformationByHandle(handle, + (FILE_INFO_BY_HANDLE_CLASS) 21, &disp, sizeof(disp)); + CloseHandle(handle); + if (ok) + DBUG_RETURN(0); + } + handle= CreateFile(name, DELETE, 0, NULL, OPEN_EXISTING, FILE_FLAG_DELETE_ON_CLOSE, NULL); if (handle != INVALID_HANDLE_VALUE) { |