diff options
| author | Steve French <stfrench@microsoft.com> | 2020-06-12 21:49:47 +0200 |
|---|---|---|
| committer | Steve French <stfrench@microsoft.com> | 2020-06-12 23:31:06 +0200 |
| commit | a7a519a4926214ba4161bc30109f4a8d69defb8d (patch) | |
| tree | 38b11973bff7ea812122d00cf591d3cb669816f4 /fs | |
| parent | cifs: fix chown and chgrp when idsfromsid mount option enabled (diff) | |
| download | linux-a7a519a4926214ba4161bc30109f4a8d69defb8d.tar.xz linux-a7a519a4926214ba4161bc30109f4a8d69defb8d.zip | |
smb3: Add debug message for new file creation with idsfromsid mount option
Pavel noticed that a debug message (disabled by default) in creating the security
descriptor context could be useful for new file creation owner fields
(as we already have for the mode) when using mount parm idsfromsid.
[38120.392272] CIFS: FYI: owner S-1-5-88-1-0, group S-1-5-88-2-0
[38125.792637] CIFS: FYI: owner S-1-5-88-1-1000, group S-1-5-88-2-1000
Also cleans up a typo in a comment
Signed-off-by: Steve French <stfrench@microsoft.com>
Reviewed-by: Pavel Shilovsky <pshilov@microsoft.com>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/cifs/inode.c | 4 | ||||
| -rw-r--r-- | fs/cifs/smb2pdu.c | 2 |
2 files changed, 4 insertions, 2 deletions
diff --git a/fs/cifs/inode.c b/fs/cifs/inode.c index 15ac63ff8ffc..583f5e4008c2 100644 --- a/fs/cifs/inode.c +++ b/fs/cifs/inode.c @@ -1148,7 +1148,7 @@ smb311_posix_get_inode_info(struct inode **inode, /* - * 4. Tweak fattr based on mount options + * 3. Tweak fattr based on mount options */ /* check for Minshall+French symlinks */ @@ -1160,7 +1160,7 @@ smb311_posix_get_inode_info(struct inode **inode, } /* - * 5. Update inode with final fattr data + * 4. Update inode with final fattr data */ if (!*inode) { diff --git a/fs/cifs/smb2pdu.c b/fs/cifs/smb2pdu.c index c6b9e617343a..2f4cdd290c46 100644 --- a/fs/cifs/smb2pdu.c +++ b/fs/cifs/smb2pdu.c @@ -2337,6 +2337,8 @@ static void setup_owner_group_sids(char *buf) sids->group.SubAuthorities[0] = cpu_to_le32(88); sids->group.SubAuthorities[1] = cpu_to_le32(2); sids->group.SubAuthorities[2] = cpu_to_le32(current_fsgid().val); + + cifs_dbg(FYI, "owner S-1-5-88-1-%d, group S-1-5-88-2-%d\n", current_fsuid().val, current_fsgid().val); } /* See MS-SMB2 2.2.13.2.2 and MS-DTYP 2.4.6 */ |