diff options
| author | Namjae Jeon <namjae.jeon@samsung.com> | 2021-03-30 14:25:35 +0900 |
|---|---|---|
| committer | Steve French <stfrench@microsoft.com> | 2021-05-10 19:15:29 -0500 |
| commit | 64b39f4a2fd293cf899dd8062c57ce3715dd7ee9 (patch) | |
| tree | 7017591533f0551bd5e93b8af29f6ad26ef9d1cb /fs/cifsd/ndr.c | |
| parent | 7cb82de3cdf2da0acd6fc3e670c7271ded37e116 (diff) | |
| download | lwn-64b39f4a2fd293cf899dd8062c57ce3715dd7ee9.tar.gz lwn-64b39f4a2fd293cf899dd8062c57ce3715dd7ee9.zip | |
cifsd: clean-up codes using chechpatch.pl --strict
Dan Carpenter suggested to run chechpatch.pl --strict on ksmbd to fix
check warnings. This patch does not fix all warnings but only things that
I can understand.
Signed-off-by: Namjae Jeon <namjae.jeon@samsung.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'fs/cifsd/ndr.c')
| -rw-r--r-- | fs/cifsd/ndr.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/fs/cifsd/ndr.c b/fs/cifsd/ndr.c index aa0cb8fc555d..1838fc2b1d7c 100644 --- a/fs/cifsd/ndr.c +++ b/fs/cifsd/ndr.c @@ -159,8 +159,9 @@ int ndr_encode_dos_attr(struct ndr *n, struct xattr_dos_attrib *da) ndr_write_int32(n, da->ea_size); ndr_write_int64(n, da->size); ndr_write_int64(n, da->alloc_size); - } else + } else { ndr_write_int64(n, da->itime); + } ndr_write_int64(n, da->create_time); if (da->version == 3) ndr_write_int64(n, da->change_time); @@ -248,15 +249,17 @@ int ndr_encode_posix_acl(struct ndr *n, struct inode *inode, /* ACL ACCESS */ ndr_write_int32(n, ref_id); ref_id += 4; - } else + } else { ndr_write_int32(n, 0); + } if (def_acl) { /* DEFAULT ACL ACCESS */ ndr_write_int32(n, ref_id); ref_id += 4; - } else + } else { ndr_write_int32(n, 0); + } ndr_write_int64(n, from_kuid(&init_user_ns, inode->i_uid)); ndr_write_int64(n, from_kgid(&init_user_ns, inode->i_gid)); |