diff options
author | Jeff Layton <jlayton@redhat.com> | 2013-04-03 11:55:03 -0400 |
---|---|---|
committer | Steve French <smfrench@gmail.com> | 2013-05-04 23:18:01 -0400 |
commit | 0124cc45113f93b52c76589dc596a026917859bb (patch) | |
tree | 543f7b35f2392e3fe24e96c940c840ef6c454304 /fs/cifs/cifssmb.c | |
parent | ad313cb86dfba27f8f2306da304974ef17c91c56 (diff) |
cifs: store the real expected sequence number in the mid
Currently, the signing routines take a pointer to a place to store the
expected sequence number for the mid response. It then stores a value
that's one below what that sequence number should be, and then adds one
to it when verifying the signature on the response.
Increment the sequence number before storing the value in the mid, and
eliminate the "+1" when checking the signature.
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/cifssmb.c')
-rw-r--r-- | fs/cifs/cifssmb.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/cifs/cifssmb.c b/fs/cifs/cifssmb.c index 728b3176984a..a58dc77cc443 100644 --- a/fs/cifs/cifssmb.c +++ b/fs/cifs/cifssmb.c | |||
@@ -1545,7 +1545,7 @@ cifs_readv_callback(struct mid_q_entry *mid) | |||
1545 | int rc = 0; | 1545 | int rc = 0; |
1546 | 1546 | ||
1547 | rc = cifs_verify_signature(&rqst, server, | 1547 | rc = cifs_verify_signature(&rqst, server, |
1548 | mid->sequence_number + 1); | 1548 | mid->sequence_number); |
1549 | if (rc) | 1549 | if (rc) |
1550 | cifs_dbg(VFS, "SMB signature verification returned error = %d\n", | 1550 | cifs_dbg(VFS, "SMB signature verification returned error = %d\n", |
1551 | rc); | 1551 | rc); |