cifs: on send failure, readjust server sequence number downward
authorJeff Layton <jlayton@redhat.com>
Wed, 3 Apr 2013 14:27:36 +0000 (10:27 -0400)
committerSteve French <smfrench@gmail.com>
Sun, 5 May 2013 03:17:56 +0000 (22:17 -0500)
If sending a call to the server fails for some reason (for instance, the
sending thread caught a signal), then we must readjust the sequence
number downward again or the next send will have it too high.

Signed-off-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Signed-off-by: Steve French <smfrench@gmail.com>
fs/cifs/smb1ops.c
fs/cifs/transport.c

index 23bcd11c81036c9d036727d64188cf870e86375c..3efdb9d5c0b8a704789070f1e0aedfaffcb9ec39 100644 (file)
@@ -61,6 +61,9 @@ send_nt_cancel(struct TCP_Server_Info *server, void *buf,
         */
        --server->sequence_number;
        rc = smb_send(server, in_buf, be32_to_cpu(in_buf->smb_buf_length));
+       if (rc < 0)
+               server->sequence_number--;
+
        mutex_unlock(&server->srv_mutex);
 
        cifs_dbg(FYI, "issued NT_CANCEL for mid %u, rc = %d\n",
index 653bf266428ee7d330fdbbf734b89854a970a497..293d2c83dfbeab99b9e23787df0b0978ce29617b 100644 (file)
@@ -522,6 +522,9 @@ cifs_call_async(struct TCP_Server_Info *server, struct smb_rqst *rqst,
        rc = smb_send_rqst(server, rqst);
        cifs_in_send_dec(server);
        cifs_save_when_sent(mid);
+
+       if (rc < 0)
+               server->sequence_number -= 2;
        mutex_unlock(&server->srv_mutex);
 
        if (rc == 0)
@@ -711,6 +714,8 @@ SendReceive2(const unsigned int xid, struct cifs_ses *ses,
        cifs_in_send_dec(ses->server);
        cifs_save_when_sent(midQ);
 
+       if (rc < 0)
+               ses->server->sequence_number -= 2;
        mutex_unlock(&ses->server->srv_mutex);
 
        if (rc < 0) {
@@ -835,6 +840,10 @@ SendReceive(const unsigned int xid, struct cifs_ses *ses,
        rc = smb_send(ses->server, in_buf, be32_to_cpu(in_buf->smb_buf_length));
        cifs_in_send_dec(ses->server);
        cifs_save_when_sent(midQ);
+
+       if (rc < 0)
+               ses->server->sequence_number -= 2;
+
        mutex_unlock(&ses->server->srv_mutex);
 
        if (rc < 0)
@@ -968,6 +977,10 @@ SendReceiveBlockingLock(const unsigned int xid, struct cifs_tcon *tcon,
        rc = smb_send(ses->server, in_buf, be32_to_cpu(in_buf->smb_buf_length));
        cifs_in_send_dec(ses->server);
        cifs_save_when_sent(midQ);
+
+       if (rc < 0)
+               ses->server->sequence_number -= 2;
+
        mutex_unlock(&ses->server->srv_mutex);
 
        if (rc < 0) {