diff options
| author |   Steve French <stfrench@microsoft.com> | 2019-10-05 10:53:58 -0500 |
|---|---|---|
| committer | Steve French <stfrench@microsoft.com> | 2019-10-08 18:19:40 -0700 |
| commit | d0959b080b1faa9741857ef607c90531b66e2fb8 (patch) | |
| tree | afd11a12c826d5deffdb80285b47e555d786313f /fs/cifs/smb2pdu.c | |
| parent | CIFS: Gracefully handle QueryInfo errors during open (diff) | |
| download | linux-dev-d0959b080b1faa9741857ef607c90531b66e2fb8.tar.xz linux-dev-d0959b080b1faa9741857ef607c90531b66e2fb8.zip | |
smb3: remove noisy debug message and minor cleanup
Message was intended only for developer temporary build
In addition cleanup two minor warnings noticed by Coverity
and a trivial change to workaround a sparse warning
Signed-off-by: Steve French <stfrench@microsoft.com>
Reviewed-by: Pavel Shilovsky <pshilov@microsoft.com>
Diffstat (limited to '')
| -rw-r--r-- | fs/cifs/smb2pdu.c | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/fs/cifs/smb2pdu.c b/fs/cifs/smb2pdu.c index fd5f2b4b7582..05149862aea4 100644 --- a/fs/cifs/smb2pdu.c +++ b/fs/cifs/smb2pdu.c @@ -751,8 +751,8 @@ add_posix_context(struct kvec *iov, unsigned int *num_iovec, umode_t mode) unsigned int num = *num_iovec; iov[num].iov_base = create_posix_buf(mode); - if (mode == -1) - cifs_dbg(VFS, "illegal mode\n"); /* BB REMOVEME */ + if (mode == ACL_NO_MODE) + cifs_dbg(FYI, "illegal mode\n"); if (iov[num].iov_base == NULL) return -ENOMEM; iov[num].iov_len = sizeof(struct create_posix); @@ -2521,11 +2521,8 @@ SMB2_open_init(struct cifs_tcon *tcon, struct smb_rqst *rqst, __u8 *oplock, return rc; } - /* TODO: add handling for the mode on create */ - if (oparms->disposition == FILE_CREATE) - cifs_dbg(VFS, "mode is 0x%x\n", oparms->mode); /* BB REMOVEME */ - - if ((oparms->disposition == FILE_CREATE) && (oparms->mode != -1)) { + if ((oparms->disposition == FILE_CREATE) && + (oparms->mode != ACL_NO_MODE)) { if (n_iov > 2) { struct create_context *ccontext = (struct create_context *)iov[n_iov-1].iov_base; |