diff options
| author | Jouni Malinen <j@w1.fi> | 2019-02-11 15:29:04 +0100 |
|---|---|---|
| committer | Johannes Berg <johannes.berg@intel.com> | 2019-02-11 16:51:18 +0100 |
| commit | 7388afe09143210f555bdd6c75035e9acc1fab96 (patch) | |
| tree | 346e6b05bc94025dfad7c2ba4ad8c1023a5035f8 /net/wireless | |
| parent | cfg80211: restore regulatory without calling userspace (diff) | |
| download | linux-7388afe09143210f555bdd6c75035e9acc1fab96.tar.xz linux-7388afe09143210f555bdd6c75035e9acc1fab96.zip | |
cfg80211: Use const more consistently in for_each_element macros
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>
Diffstat (limited to 'net/wireless')
| -rw-r--r-- | net/wireless/nl80211.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/wireless/nl80211.c b/net/wireless/nl80211.c index 5d85f6032f84..80878b431584 100644 --- a/net/wireless/nl80211.c +++ b/net/wireless/nl80211.c @@ -205,7 +205,7 @@ static int validate_ie_attr(const struct nlattr *attr, { const u8 *data = nla_data(attr); unsigned int len = nla_len(attr); - struct element *elem; + const struct element *elem; for_each_element(elem, data, len) { /* nothing */ |