diff options
| author | Gustavo A. R. Silva <gustavoars@kernel.org> | 2020-11-20 12:24:14 -0600 |
|---|---|---|
| committer | Steve French <stfrench@microsoft.com> | 2020-12-13 19:12:07 -0600 |
| commit | 21ac58f495849844dfb4479cda1427bf42493262 (patch) | |
| tree | 16012c934293b4a2f0c54730acc83a82cb1c4f7f /fs/cifs | |
| parent | 2c85ebc57b3e1817b6ce1a6b703928e113a90442 (diff) | |
cifs: Fix fall-through warnings for Clang
In preparation to enable -Wimplicit-fallthrough for Clang, fix multiple
warnings by explicitly adding multiple break/goto statements instead of
just letting the code fall through to the next case.
Link: https://github.com/KSPP/linux/issues/115
Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'fs/cifs')
| -rw-r--r-- | fs/cifs/inode.c | 1 | ||||
| -rw-r--r-- | fs/cifs/sess.c | 1 | ||||
| -rw-r--r-- | fs/cifs/smbdirect.c | 1 |
3 files changed, 3 insertions, 0 deletions
diff --git a/fs/cifs/inode.c b/fs/cifs/inode.c index 9ee5f304592f..ac01f9684b39 100644 --- a/fs/cifs/inode.c +++ b/fs/cifs/inode.c @@ -771,6 +771,7 @@ cifs_get_file_info(struct file *filp) */ rc = 0; CIFS_I(inode)->time = 0; + goto cgfi_exit; default: goto cgfi_exit; } diff --git a/fs/cifs/sess.c b/fs/cifs/sess.c index de564368a887..6c2c42f8d893 100644 --- a/fs/cifs/sess.c +++ b/fs/cifs/sess.c @@ -812,6 +812,7 @@ cifs_select_sectype(struct TCP_Server_Info *server, enum securityEnum requested) return NTLMv2; if (global_secflags & CIFSSEC_MAY_NTLM) return NTLM; + break; default: break; } diff --git a/fs/cifs/smbdirect.c b/fs/cifs/smbdirect.c index b029ed31ef91..10dfe5006792 100644 --- a/fs/cifs/smbdirect.c +++ b/fs/cifs/smbdirect.c @@ -246,6 +246,7 @@ smbd_qp_async_error_upcall(struct ib_event *event, void *context) case IB_EVENT_CQ_ERR: case IB_EVENT_QP_FATAL: smbd_disconnect_rdma_connection(info); + break; default: break; |