diff options
| author | Jeff <jlayton@redhat.com> | 2007-07-06 21:10:07 +0000 |
|---|---|---|
| committer | Steve French <sfrench@us.ibm.com> | 2007-07-06 21:10:07 +0000 |
| commit | 38c10a1ddb24b9fa9f25b6c21d0390a9723ae6d6 (patch) | |
| tree | aa6dbc101c0c3a308c778fa5d7708c21b86499ee /fs/cifs/cifssmb.c | |
| parent | d38d8c74c7cdfae910f9cd0ac5e1b7ba2d8a2fb2 (diff) | |
| download | lwn-38c10a1ddb24b9fa9f25b6c21d0390a9723ae6d6.tar.gz lwn-38c10a1ddb24b9fa9f25b6c21d0390a9723ae6d6.zip | |
[CIFS] Mount should fail if server signing off but client mount option requires it
Currently, if mount with a signing-enabled sec= option (e.g.
sec=ntlmi), the kernel does a warning printk if the server doesn't
support signing, and then proceeds without signatures.
This is probably OK for people that think to look at the ring buffer,
but seems wrong to me. If someone explicitly requests signing, we
should error out if that request can't be satisfied. They can then
reattempt the mount without signing if that's ok.
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'fs/cifs/cifssmb.c')
| -rw-r--r-- | fs/cifs/cifssmb.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/cifs/cifssmb.c b/fs/cifs/cifssmb.c index 4a2458e78784..b4916eb6fd43 100644 --- a/fs/cifs/cifssmb.c +++ b/fs/cifs/cifssmb.c @@ -645,11 +645,12 @@ signing_check: ~(SECMODE_SIGN_ENABLED | SECMODE_SIGN_REQUIRED); } else if ((secFlags & CIFSSEC_MUST_SIGN) == CIFSSEC_MUST_SIGN) { /* signing required */ - cFYI(1, ("Must sign - segFlags 0x%x", secFlags)); + cFYI(1, ("Must sign - secFlags 0x%x", secFlags)); if ((server->secMode & (SECMODE_SIGN_ENABLED | SECMODE_SIGN_REQUIRED)) == 0) { cERROR(1, ("signing required but server lacks support")); + rc = -EOPNOTSUPP; } else server->secMode |= SECMODE_SIGN_REQUIRED; } else { |