Silence compiler warnings about out of array bounds indexes
authorJouni Malinen <j@w1.fi>
Thu, 5 Jun 2008 14:50:53 +0000 (17:50 +0300)
committerJouni Malinen <j@w1.fi>
Thu, 5 Jun 2008 14:50:53 +0000 (17:50 +0300)
struct wpa_ie_hdr had separate fields for 24-bit OUI and 8-bit oui_type
for WPA/RSN selectors. {WPA,RSN}_SELECTOR_{GET,PUT} access these four
octets through oui and the "out-of-bounds" access for the fourth octet is
actually reading/writing oui_type. This works fine, but some tools complain
about the array bounds "failure". Since oui_type is never accessed
separately, the simplest fix is to just combine these into a single 4-octet
field.

src/common/wpa_common.h

index 936df73..9b9e39e 100644 (file)
@@ -188,8 +188,7 @@ struct wpa_ptk {
 struct wpa_ie_hdr {
        u8 elem_id;
        u8 len;
-       u8 oui[3];
-       u8 oui_type;
+       u8 oui[4]; /* 24-bit OUI followed by 8-bit OUI type */
        u8 version[2]; /* little endian */
 } STRUCT_PACKED;