rtlwifi: Fix possible unaligned array in ether_addr_copy()
authorLarry Finger <Larry.Finger@lwfinger.net>
Wed, 8 Oct 2014 17:44:55 +0000 (12:44 -0500)
committerJohn W. Linville <linville@tuxdriver.com>
Wed, 8 Oct 2014 19:24:16 +0000 (15:24 -0400)
Two macros used to copy BSSID information use ether_addr_copy(), thus
the arrays must be 2-byte aligned. In one case, the array could become
unaligned if the struct containing it were changed. Use the __unaligned(2)
attribute to retain the necessary alignment. In addition, the magic number
used to specify the size of the array is replaced by ETH_ALEN.

Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
Acked-by: David S. Miller <davem@davemloft.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/rtlwifi/wifi.h

index 976667a..6866dcf 100644 (file)
@@ -1370,7 +1370,7 @@ struct rtl_mac {
        bool rdg_en;
 
        /*AP*/
-       u8 bssid[6];
+       u8 bssid[ETH_ALEN] __aligned(2);
        u32 vendor;
        u8 mcs[16];     /* 16 bytes mcs for HT rates. */
        u32 basic_rates; /* b/g rates */