diff options
author | Jeff Layton <jlayton@redhat.com> | 2011-10-11 06:41:32 -0400 |
---|---|---|
committer | Steve French <smfrench@gmail.com> | 2011-10-12 23:41:36 -0500 |
commit | 376b43f41c8b9315f7efdf085d214b6024337381 (patch) | |
tree | 0a5e1959a71d2b645faf731ae062c7a27fdc8421 /fs/cifs/transport.c | |
parent | c974befa402b5eb2ed115b3083b5a46a4be85a9f (diff) | |
download | lwn-376b43f41c8b9315f7efdf085d214b6024337381.tar.gz lwn-376b43f41c8b9315f7efdf085d214b6024337381.zip |
cifs: clean up checkSMB
The variable names in this function are so ambiguous that it's very
difficult to know what it's doing. Rename them to make it a bit more
clear.
Also, remove a redundant length check. cifsd checks to make sure that
the rfclen isn't larger than the maximum frame size when it does the
receive.
Finally, change checkSMB to return a real error code (-EIO) when
it finds an error. That will help simplify some coming changes in the
callers.
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve French <smfrench@gmail.com>
Diffstat (limited to 'fs/cifs/transport.c')
0 files changed, 0 insertions, 0 deletions