2020-07-14 10:56:52

by Yi Wang

Subject: [PATCH] cifs: Remove the superfluous break

From: Liao Pingfang <[email protected]>

Remove the superfuous break, as there is a 'return' before it.

Signed-off-by: Liao Pingfang <[email protected]>
Signed-off-by: Yi Wang <[email protected]>
---
fs/cifs/sess.c | 1 -
1 file changed, 1 deletion(-)

diff --git a/fs/cifs/sess.c b/fs/cifs/sess.c
index 5d05bd2..6708ab0 100644
--- a/fs/cifs/sess.c
+++ b/fs/cifs/sess.c
@@ -1705,7 +1705,6 @@ static int select_sec(struct cifs_ses *ses, struct sess_data *sess_data)
#else
cifs_dbg(VFS, "Kerberos negotiated but upcall support disabled!\n");
return -ENOSYS;
- break;
#endif /* CONFIG_CIFS_UPCALL */
case RawNTLMSSP:
sess_data->func = sess_auth_rawntlmssp_negotiate;
--
2.9.5

2020-07-15 04:21:06

by Steve French

Subject: Re: [PATCH] cifs: Remove the superfluous break

merged into cifs-2.6.git for-next

thx

On Tue, Jul 14, 2020 at 6:14 AM Yi Wang <[email protected]> wrote:
>
> From: Liao Pingfang <[email protected]>
>
> Remove the superfuous break, as there is a 'return' before it.
>
> Signed-off-by: Liao Pingfang <[email protected]>
> Signed-off-by: Yi Wang <[email protected]>
> ---
> fs/cifs/sess.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/fs/cifs/sess.c b/fs/cifs/sess.c
> index 5d05bd2..6708ab0 100644
> --- a/fs/cifs/sess.c
> +++ b/fs/cifs/sess.c
> @@ -1705,7 +1705,6 @@ static int select_sec(struct cifs_ses *ses, struct sess_data *sess_data)
> #else
> cifs_dbg(VFS, "Kerberos negotiated but upcall support disabled!\n");
> return -ENOSYS;
> - break;
> #endif /* CONFIG_CIFS_UPCALL */
> case RawNTLMSSP:
> sess_data->func = sess_auth_rawntlmssp_negotiate;
> --
> 2.9.5
>


--
Thanks,

Steve