diff options
| author | Jeff Layton <jlayton@redhat.com> | 2011-10-19 15:29:59 -0400 |
|---|---|---|
| committer | Jeff Layton <jlayton@redhat.com> | 2011-10-19 15:29:59 -0400 |
| commit | 2ab2593f4b8953ff951f5531e695e487dfe0b51f (patch) | |
| tree | ad5be299146015bea9f83c066ebf11a2b26454ff | |
| parent | 44d22d846fdc7c3e688fc1ff5ae6d06d08bb5656 (diff) | |
| download | op-kernel-dev-2ab2593f4b8953ff951f5531e695e487dfe0b51f.zip op-kernel-dev-2ab2593f4b8953ff951f5531e695e487dfe0b51f.tar.gz | |
cifs: fix protocol definition for READ_RSP
There is no pad, and it simplifies the code to remove the "Data" field.
None of the existing code relies on these fields, or on the READ_RSP
being a particular length.
Reviewed-and-Tested-by: Pavel Shilovsky <piastry@etersoft.ru>
Signed-off-by: Jeff Layton <jlayton@redhat.com>
| -rw-r--r-- | fs/cifs/cifspdu.h | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/fs/cifs/cifspdu.h b/fs/cifs/cifspdu.h index de3aa28..3c6ef34 100644 --- a/fs/cifs/cifspdu.h +++ b/fs/cifs/cifspdu.h @@ -1089,9 +1089,7 @@ typedef struct smb_com_read_rsp { __le16 DataLengthHigh; __u64 Reserved2; __u16 ByteCount; - __u8 Pad; /* BB check for whether padded to DWORD - boundary and optimum performance here */ - char Data[1]; + /* read response data immediately follows */ } __attribute__((packed)) READ_RSP; typedef struct locking_andx_range { |
