diff options
author | Jeff Layton <jlayton@redhat.com> | 2013-04-03 10:27:36 -0400 |
---|---|---|
committer | Steve French <smfrench@gmail.com> | 2013-05-04 23:17:56 -0400 |
commit | ad313cb86dfba27f8f2306da304974ef17c91c56 (patch) | |
tree | b6fbc7b3f6d7448264c7db49ef82f300c2699d3e /fs/cifs/transport.c | |
parent | ce6c44e44e43f0cef12030de0a8f0814914d47dd (diff) |
cifs: on send failure, readjust server sequence number downward
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>
Diffstat (limited to 'fs/cifs/transport.c')
-rw-r--r-- | fs/cifs/transport.c | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/fs/cifs/transport.c b/fs/cifs/transport.c index 653bf266428e..293d2c83dfbe 100644 --- a/fs/cifs/transport.c +++ b/fs/cifs/transport.c | |||
@@ -522,6 +522,9 @@ cifs_call_async(struct TCP_Server_Info *server, struct smb_rqst *rqst, | |||
522 | rc = smb_send_rqst(server, rqst); | 522 | rc = smb_send_rqst(server, rqst); |
523 | cifs_in_send_dec(server); | 523 | cifs_in_send_dec(server); |
524 | cifs_save_when_sent(mid); | 524 | cifs_save_when_sent(mid); |
525 | |||
526 | if (rc < 0) | ||
527 | server->sequence_number -= 2; | ||
525 | mutex_unlock(&server->srv_mutex); | 528 | mutex_unlock(&server->srv_mutex); |
526 | 529 | ||
527 | if (rc == 0) | 530 | if (rc == 0) |
@@ -711,6 +714,8 @@ SendReceive2(const unsigned int xid, struct cifs_ses *ses, | |||
711 | cifs_in_send_dec(ses->server); | 714 | cifs_in_send_dec(ses->server); |
712 | cifs_save_when_sent(midQ); | 715 | cifs_save_when_sent(midQ); |
713 | 716 | ||
717 | if (rc < 0) | ||
718 | ses->server->sequence_number -= 2; | ||
714 | mutex_unlock(&ses->server->srv_mutex); | 719 | mutex_unlock(&ses->server->srv_mutex); |
715 | 720 | ||
716 | if (rc < 0) { | 721 | if (rc < 0) { |
@@ -835,6 +840,10 @@ SendReceive(const unsigned int xid, struct cifs_ses *ses, | |||
835 | rc = smb_send(ses->server, in_buf, be32_to_cpu(in_buf->smb_buf_length)); | 840 | rc = smb_send(ses->server, in_buf, be32_to_cpu(in_buf->smb_buf_length)); |
836 | cifs_in_send_dec(ses->server); | 841 | cifs_in_send_dec(ses->server); |
837 | cifs_save_when_sent(midQ); | 842 | cifs_save_when_sent(midQ); |
843 | |||
844 | if (rc < 0) | ||
845 | ses->server->sequence_number -= 2; | ||
846 | |||
838 | mutex_unlock(&ses->server->srv_mutex); | 847 | mutex_unlock(&ses->server->srv_mutex); |
839 | 848 | ||
840 | if (rc < 0) | 849 | if (rc < 0) |
@@ -968,6 +977,10 @@ SendReceiveBlockingLock(const unsigned int xid, struct cifs_tcon *tcon, | |||
968 | rc = smb_send(ses->server, in_buf, be32_to_cpu(in_buf->smb_buf_length)); | 977 | rc = smb_send(ses->server, in_buf, be32_to_cpu(in_buf->smb_buf_length)); |
969 | cifs_in_send_dec(ses->server); | 978 | cifs_in_send_dec(ses->server); |
970 | cifs_save_when_sent(midQ); | 979 | cifs_save_when_sent(midQ); |
980 | |||
981 | if (rc < 0) | ||
982 | ses->server->sequence_number -= 2; | ||
983 | |||
971 | mutex_unlock(&ses->server->srv_mutex); | 984 | mutex_unlock(&ses->server->srv_mutex); |
972 | 985 | ||
973 | if (rc < 0) { | 986 | if (rc < 0) { |