| commit | 61e16b8c3e8f5657403acabbb5d4eb42e2e6b0a1 | |
| author | Jeff Layton <jlayton@redhat.com> | |
| Wed, 27 Apr 2011 17:25:51 +0000 (27 13:25 -0400) | ||
| committer | Greg Kroah-Hartman <gregkh@suse.de> | |
| Sat, 21 May 2011 22:13:02 +0000 (21 15:13 -0700) | ||
| tree | cd9ca6bf3128fac96263cb9c3dcf721070124062 | treesnapshot (tar.gz zip) |
| parent | 96bea36718ae2f617dbbab2587acd6c7c0c9d3df | commitdiff |
cifs: check for bytes_remaining going to zero in CIFS_SessSetup
commit fcda7f4578bbf9717444ca6da8a421d21489d078 upstream.
It's possible that when we go to decode the string area in the
SESSION_SETUP response, that bytes_remaining will be 0. Decrementing it at
that point will mean that it can go "negative" and wrap. Check for a
bytes_remaining value of 0, and don't try to decode the string area if
that's the case.
Reported-and-Acked-by: David Howells <dhowells@redhat.com>
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
commit fcda7f4578bbf9717444ca6da8a421d21489d078 upstream.
It's possible that when we go to decode the string area in the
SESSION_SETUP response, that bytes_remaining will be 0. Decrementing it at
that point will mean that it can go "negative" and wrap. Check for a
bytes_remaining value of 0, and don't try to decode the string area if
that's the case.
Reported-and-Acked-by: David Howells <dhowells@redhat.com>
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
| fs/cifs/sess.c | diffblobblamehistory |