Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use fstatfs to detect whether current file system supports shared locks for files opened for writing #55256

Merged
merged 16 commits into from
Jul 9, 2021
Merged

use fstatfs to detect whether current file system supports shared locks for files opened for writing #55256

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
71d131a
use fstatfs to detect whether current file system supports shared loc…
adamsitnik Jul 7, 2021
7980cc8
improve comments
adamsitnik Jul 7, 2021
97c1544
cast in explicit way
adamsitnik Jul 7, 2021
5d95af8
hopefully fix the BSD build(s)
adamsitnik Jul 7, 2021
d64374f
fix OSX build
adamsitnik Jul 7, 2021
c687e6a
assume we should not acquire the lock if we don't know the File System
adamsitnik Jul 7, 2021
9c481ed
try to fix the maccatalyst build
adamsitnik Jul 7, 2021
1c890ec
always use fstatfs as it's available on every platform and can get us…
adamsitnik Jul 8, 2021
04eced6
handle SMB2_SUPER_MAGIC as well (#53182)
adamsitnik Jul 8, 2021
79b6def
introduce a flag that allows to disable file locking on Unix
adamsitnik Jul 8, 2021
ba6d027
Emscripten does not support file locking, so there is no need to chec…
adamsitnik Jul 8, 2021
731e65c
some polishing
adamsitnik Jul 8, 2021
d31f6e1
Merge remote-tracking branch 'upstream/main' into superMagic
adamsitnik Jul 8, 2021
c752b93
reuse the UnixFileSystemTypes enum, move it to a separate file so Cor…
adamsitnik Jul 9, 2021
7443a5e
add SMB2_SUPER_MAGIC to the enum
adamsitnik Jul 9, 2021
1e3878c
don't try to unlock the file if it has not been locked
adamsitnik Jul 9, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
handle SMB2_SUPER_MAGIC as well (#53182)
  • Loading branch information
@adamsitnik
adamsitnik committed Jul 8, 2021
commit 04eced6414f39399ac8783aaa2d9ff2725efa4b7
1 change: 1 addition & 0 deletions src/libraries/System.Private.CoreLib/src/Microsoft/Win32/SafeHandles/SafeFileHandle.Unix.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -337,6 +337,7 @@ private bool CanLockTheFile(Interop.Sys.LockOperations lockOperation, FileAccess
case 0x6969: // NFS_SUPER_MAGIC
case 0xFF534D42: // CIFS_MAGIC_NUMBER
case 0x517B: // SMB_SUPER_MAGIC
case 0xFE534D42: // SMB2_SUPER_MAGIC (#53182)
return false; // LOCK_SH is not OK when writing to NFS, CIFS or SMB
case -1: // error
return false; // assume we should not acquire the lock if we don't know the File System
Expand Down