4.13-stable review patch.  If anyone has any objections, please let me know.

------------------

From: Steve French <smfre...@gmail.com>

commit 590d08d3da45e9fed423b08ab38d71886c07abc8 upstream.

Multi-dialect negotiate patch had a minor endian error.

Signed-off-by: Steve French <smfre...@gmail.com>
Reviewed-by: Ronnie Sahlberg <lsahl...@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gre...@linuxfoundation.org>

---
 fs/cifs/smb2pdu.c |    5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

--- a/fs/cifs/smb2pdu.c
+++ b/fs/cifs/smb2pdu.c
@@ -558,10 +558,11 @@ SMB2_negotiate(const unsigned int xid, s
                        /* ops set to 3.0 by default for default so update */
                        ses->server->ops = &smb21_operations;
                }
-       } else if (rsp->DialectRevision != ses->server->vals->protocol_id) {
+       } else if (le16_to_cpu(rsp->DialectRevision) !=
+                               ses->server->vals->protocol_id) {
                /* if requested single dialect ensure returned dialect matched 
*/
                cifs_dbg(VFS, "Illegal 0x%x dialect returned: not requested\n",
-                       cpu_to_le16(rsp->DialectRevision));
+                       le16_to_cpu(rsp->DialectRevision));
                return -EIO;
        }
 


Reply via email to