Skip to content

Commit 8204d97

Browse files
committed
compat/mingw.c: do not warn when failing to get owner
In the case of Git for Windows (say, in a Git Bash window) running in a Windows Subsystem for Linux (WSL) directory, the GetNamedSecurityInfoW() call in is_path_owned_By_current_side() returns an error code other than ERROR_SUCCESS. This is consistent behavior across this boundary. In these cases, the owner would always be different because the WSL owner is a different entity than the Windows user. The change here is to suppress the error message that looks like this: error: failed to get owner for '//wsl.localhost/...' (1) Before this change, this warning happens for every Git command, regardless of whether the directory is marked with safe.directory. Signed-off-by: Derrick Stolee <[email protected]>
1 parent 1d0b139 commit 8204d97

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

compat/mingw.c

+1-3
Original file line numberDiff line numberDiff line change
@@ -3509,9 +3509,7 @@ int is_path_owned_by_current_sid(const char *path)
35093509
DACL_SECURITY_INFORMATION,
35103510
&sid, NULL, NULL, NULL, &descriptor);
35113511

3512-
if (err != ERROR_SUCCESS)
3513-
error(_("failed to get owner for '%s' (%ld)"), path, err);
3514-
else if (sid && IsValidSid(sid)) {
3512+
if (err == ERROR_SUCCESS && sid && IsValidSid(sid)) {
35153513
/* Now, verify that the SID matches the current user's */
35163514
static PSID current_user_sid;
35173515
BOOL is_member;

0 commit comments

Comments
 (0)