Skip to content

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

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

Merged
adamsitnik 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
Emscripten does not support file locking, so there is no need to chec… 
…k the file system type
  • Loading branch information
@adamsitnik
adamsitnik committed Jul 8, 2021
commit ba6d027297107d49d32c587936c777b79a1fddde
6 changes: 5 additions & 1 deletion 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 @@ -328,7 +328,11 @@ private bool CanLockTheFile(Interop.Sys.LockOperations lockOperation, FileAccess
{
Debug.Assert(lockOperation == Interop.Sys.LockOperations.LOCK_EX || lockOperation == Interop.Sys.LockOperations.LOCK_SH);

if (DisableFileLocking)
if (OperatingSystem.IsBrowser())
{
return false; // #40065: Emscripten does not support file locking
}
else if (DisableFileLocking)
{
return false;
}
Expand Down