diff options
author | Jouni Malinen <j@w1.fi> | 2019-02-11 16:29:04 +0200 |
---|---|---|
committer | Todd Kjos <tkjos@google.com> | 2020-08-05 15:10:47 -0700 |
commit | aabaaab3cfc40ed6588e04aeb02269ad7d7f73cc (patch) | |
tree | 923cd7705417dae9154a883cb6a3f9887911bd9d | |
parent | ea631b4df7a44d9dfd89ed57cd05484e9e44eea8 (diff) |
cfg80211: Use const more consistently in for_each_element macrosASB-2020-08-05_3.18-o-mr1
commit 7388afe09143210f555bdd6c75035e9acc1fab96 upstream.
Enforce the first argument to be a correct type of a pointer to struct
element and avoid unnecessary typecasts from const to non-const pointers
(the change in validate_ie_attr() is needed to make this part work). In
addition, avoid signed/unsigned comparison within for_each_element() and
mark struct element packed just in case.
Signed-off-by: Jouni Malinen <j@w1.fi>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
Change-Id: Ib151a7fbf7e474571e2906b9e4f351a20a3640c2
-rw-r--r-- | include/linux/ieee80211.h | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/include/linux/ieee80211.h b/include/linux/ieee80211.h index 6c6e9259ba5f..1da9993ed54e 100644 --- a/include/linux/ieee80211.h +++ b/include/linux/ieee80211.h @@ -2453,16 +2453,16 @@ struct element { u8 id; u8 datalen; u8 data[]; -}; +} __packed; /* element iteration helpers */ -#define for_each_element(element, _data, _datalen) \ - for (element = (void *)(_data); \ - (u8 *)(_data) + (_datalen) - (u8 *)element >= \ - sizeof(*element) && \ - (u8 *)(_data) + (_datalen) - (u8 *)element >= \ - sizeof(*element) + element->datalen; \ - element = (void *)(element->data + element->datalen)) +#define for_each_element(_elem, _data, _datalen) \ + for (_elem = (const struct element *)(_data); \ + (const u8 *)(_data) + (_datalen) - (const u8 *)_elem >= \ + (int)sizeof(*_elem) && \ + (const u8 *)(_data) + (_datalen) - (const u8 *)_elem >= \ + (int)sizeof(*_elem) + _elem->datalen; \ + _elem = (const struct element *)(_elem->data + _elem->datalen)) #define for_each_element_id(element, _id, data, datalen) \ for_each_element(element, data, datalen) \ @@ -2499,7 +2499,7 @@ struct element { static inline bool for_each_element_completed(const struct element *element, const void *data, size_t datalen) { - return (u8 *)element == (u8 *)data + datalen; + return (const u8 *)element == (const u8 *)data + datalen; } #endif /* LINUX_IEEE80211_H */ |