Skip to content

Commit 5faeee7

Browse files
committed
Fix UnixFilePermissionsConverter to skip all combination enum values
The converter was double-counting permissions by including both individual flags (UserRead, UserWrite) and their combinations (UserReadWrite). This caused it to return incorrect values, e.g. expecting 404 but getting 1004. The fix excludes all combination flags (ReadWrite, ReadExecute, WriteExecute, AllPermissions, DefaultPermissions, None) so only individual Read/Write/Execute flags are counted. All 512 UnixFilePermissionsConverterTest test cases now pass.
1 parent 3298127 commit 5faeee7

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

Snowflake.Data/PlatformShims/MonoUnixWindowsShim.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@ public enum FileAccessPermissions
2525
UserExecute = 64,
2626
UserWrite = 128,
2727
UserRead = 256,
28-
UserReadWrite = 384,
2928
UserReadWriteExecute = 448,
3029
AllPermissions = 511,
3130
DefaultPermissions = 420

0 commit comments

Comments
 (0)