wifi: cfg80211: make cfg80211_auth_request::key_idx signed
authorJohannes Berg <johannes.berg@intel.com>
Wed, 22 Jun 2022 08:17:23 +0000 (10:17 +0200)
committerJohannes Berg <johannes.berg@intel.com>
Fri, 15 Jul 2022 09:43:14 +0000 (11:43 +0200)
We might assign -1 to it in some cases when key is NULL,
which means the key_idx isn't used but can lead to a
warning from static checkers such as smatch.

Make the struct member signed simply to avoid that, we
only need a range of -1..3 anyway.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
include/net/cfg80211.h

index c904cbd..a4e2cb2 100644 (file)
@@ -2773,7 +2773,8 @@ struct cfg80211_auth_request {
        size_t ie_len;
        enum nl80211_auth_type auth_type;
        const u8 *key;
-       u8 key_len, key_idx;
+       u8 key_len;
+       s8 key_idx;
        const u8 *auth_data;
        size_t auth_data_len;
        s8 link_id;