usb/storage: remove Filler member from struct bulk_cs_wrap

As Alan Stern pointed out this member has nothing to do with the Command
Status Wrapper (CSW) as specified by the Universal Serial Bus Mass
Storage Class Bulk-Only Transport rev 1.0. It defines the structure
without the additional 18 filler bytes and defines the total size of the
struct to exactly 13 bytes. Larger responses should be dropped. All
in-tree users use a defines instead of sizeof() of this struct as far I
can tell.

Cc: Alan Stern <stern@rowland.harvard.edu>
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/include/linux/usb/storage.h b/include/linux/usb/storage.h
index 4de58b15fad63b082d8c410493e36697c34dcf05..cb33fff2ba0be33f3aa6b764e78b402f948e9c4d 100644 (file)
--- a/include/linux/usb/storage.h
+++ b/include/linux/usb/storage.h
@@ -71,7 +71,6 @@ struct bulk_cs_wrap {
        __u32   Tag;            /* same as original command */
        __le32  Residue;        /* amount not transferred */
        __u8    Status;         /* see below */
-       __u8    Filler[18];
 };
 
 #define US_BULK_CS_WRAP_LEN    13