diff mbox series

[CIFS] print warning once if mounting with vers=1.0

Message ID CAH2r5mt8Zg5ZT4hCKQr2J-Z3pZXpgYA9Gbv18VzPkWB3B7ZXwg@mail.gmail.com
State New
Headers show
Series [CIFS] print warning once if mounting with vers=1.0 | expand

Commit Message

Steve French Feb. 28, 2020, 7:37 a.m. UTC 
We really, really don't want people using insecure dialects
    unless they realize what they are doing ...

    Add print once warning if mounting with vers=1.0 (older SMB1/CIFS
    dialect) instead of the default (SMB2.1 or later, typically
    SMB3.1.1).
diff mbox series

Patch

From a413f3274d270209f9f623bca301dbb8a772df63 Mon Sep 17 00:00:00 2001
From: Steve French <stfrench@microsoft.com>
Date: Fri, 28 Feb 2020 01:32:37 -0600
Subject: [PATCH] cifs: print warning once if mounting with vers=1.0

We really, really don't want people using insecure dialects
unless they realize what they are doing ...

Add print once warning if mounting with vers=1.0 (older SMB1/CIFS
dialect) instead of the default (SMB2.1 or later, typically
SMB3.1.1).

Signed-off-by: Steve French <stfrench@microsoft.com>
---
 fs/cifs/connect.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c
index f4d12b79ceed..27577d71d942 100644
--- a/fs/cifs/connect.c
+++ b/fs/cifs/connect.c
@@ -1471,6 +1471,9 @@  cifs_parse_smb_version(char *value, struct smb_vol *vol, bool is_smb3)
 			cifs_dbg(VFS, "vers=1.0 (cifs) not permitted when mounting with smb3\n");
 			return 1;
 		}
+		printk_once(KERN_WARNING "Use of the less secure dialect "
+			   "vers=1.0 is not recommended unless required for "
+			   "access to very old servers\n");
 		vol->ops = &smb1_operations;
 		vol->vals = &smb1_values;
 		break;
-- 
2.20.1