diff options
author | Steve French <smfrench@gmail.com> | 2017-09-19 12:43:47 -0400 |
---|---|---|
committer | Steve French <smfrench@gmail.com> | 2017-09-20 02:38:44 -0400 |
commit | 590d08d3da45e9fed423b08ab38d71886c07abc8 (patch) | |
tree | be088b886bd31a2f3a41a0c772a809f88c9d083a | |
parent | 23586b66d84ba3184b8820277f3fc42761640f87 (diff) |
SMB3: Fix endian warning
Multi-dialect negotiate patch had a minor endian error.
Signed-off-by: Steve French <smfrench@gmail.com>
Reviewed-by: Ronnie Sahlberg <lsahlber@redhat.com>
CC: Stable <stable@vger.kernel.org> # 4.13+
-rw-r--r-- | fs/cifs/smb2pdu.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/cifs/smb2pdu.c b/fs/cifs/smb2pdu.c index b0eaebe627e9..b4c58a1db1ae 100644 --- a/fs/cifs/smb2pdu.c +++ b/fs/cifs/smb2pdu.c | |||
@@ -570,10 +570,11 @@ SMB2_negotiate(const unsigned int xid, struct cifs_ses *ses) | |||
570 | /* ops set to 3.0 by default for default so update */ | 570 | /* ops set to 3.0 by default for default so update */ |
571 | ses->server->ops = &smb21_operations; | 571 | ses->server->ops = &smb21_operations; |
572 | } | 572 | } |
573 | } else if (rsp->DialectRevision != ses->server->vals->protocol_id) { | 573 | } else if (le16_to_cpu(rsp->DialectRevision) != |
574 | ses->server->vals->protocol_id) { | ||
574 | /* if requested single dialect ensure returned dialect matched */ | 575 | /* if requested single dialect ensure returned dialect matched */ |
575 | cifs_dbg(VFS, "Illegal 0x%x dialect returned: not requested\n", | 576 | cifs_dbg(VFS, "Illegal 0x%x dialect returned: not requested\n", |
576 | cpu_to_le16(rsp->DialectRevision)); | 577 | le16_to_cpu(rsp->DialectRevision)); |
577 | return -EIO; | 578 | return -EIO; |
578 | } | 579 | } |
579 | 580 | ||