mirror of
https://github.com/BobTheBlinker/android_kernel_motorola_sm6375.git
synced 2026-10-05 19:31:57 -04:00
wifi: cfg80211: Increase akm_suites array size in
cfg80211_crypto_settings Increase akm_suites array size in struct cfg80211_crypto_settings to 10 and advertise the capability to userspace. This allows userspace to send more than two AKMs to driver in netlink commands such as NL80211_CMD_CONNECT. This capability is needed for implementing WPA3-Personal transition mode correctly with any driver that handles roaming internally. Currently, the possible AKMs for multi-AKM connect can include PSK, PSK-SHA-256, SAE, FT-PSK and FT-SAE. Since the count is already 5, increasing the akm_suites array size to 10 should be reasonable for future usecases. Signed-off-by: Veerendranath Jakkam <quic_vjakkam@quicinc.com> Link: https://lore.kernel.org/r/1653312358-12321-1-git-send-email-quic_vjakkam@quicinc.com Signed-off-by: Johannes Berg <johannes.berg@intel.com> Git-commit: ecad3b0b99bff7247a11f8c7cb19ac9b0cb28b09 Git-repo: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git. Change-Id: I4ce3de12ec85e11cf30d0b26e2324bc62ab5e73e CRs-Fixed: 3799177 Signed-off-by: Srikanth Marepalli <quic_srimarep@quicinc.com>
This commit is contained in:
parent
ad064077b5
commit
525fb1b48f
5 changed files with 45 additions and 6 deletions
|
|
@ -222,6 +222,7 @@ int qtnf_cmd_send_start_ap(struct qtnf_vif *vif,
|
|||
struct qlink_auth_encr *aen;
|
||||
int ret;
|
||||
int i;
|
||||
int n;
|
||||
|
||||
if (!qtnf_cmd_start_ap_can_fit(vif, s))
|
||||
return -E2BIG;
|
||||
|
|
@ -253,8 +254,9 @@ int qtnf_cmd_send_start_ap(struct qtnf_vif *vif,
|
|||
for (i = 0; i < QLINK_MAX_NR_CIPHER_SUITES; i++)
|
||||
aen->ciphers_pairwise[i] =
|
||||
cpu_to_le32(s->crypto.ciphers_pairwise[i]);
|
||||
aen->n_akm_suites = cpu_to_le32(s->crypto.n_akm_suites);
|
||||
for (i = 0; i < QLINK_MAX_NR_AKM_SUITES; i++)
|
||||
n = min(QLINK_MAX_NR_AKM_SUITES, s->crypto.n_akm_suites);
|
||||
aen->n_akm_suites = cpu_to_le32(n);
|
||||
for (i = 0; i < n; i++)
|
||||
aen->akm_suites[i] = cpu_to_le32(s->crypto.akm_suites[i]);
|
||||
aen->control_port = s->crypto.control_port;
|
||||
aen->control_port_no_encrypt = s->crypto.control_port_no_encrypt;
|
||||
|
|
@ -2200,6 +2202,7 @@ int qtnf_cmd_send_connect(struct qtnf_vif *vif,
|
|||
struct qlink_auth_encr *aen;
|
||||
int ret;
|
||||
int i;
|
||||
int n;
|
||||
u32 connect_flags = 0;
|
||||
|
||||
cmd_skb = qtnf_cmd_alloc_new_cmdskb(vif->mac->macid, vif->vifid,
|
||||
|
|
@ -2256,9 +2259,10 @@ int qtnf_cmd_send_connect(struct qtnf_vif *vif,
|
|||
aen->ciphers_pairwise[i] =
|
||||
cpu_to_le32(sme->crypto.ciphers_pairwise[i]);
|
||||
|
||||
aen->n_akm_suites = cpu_to_le32(sme->crypto.n_akm_suites);
|
||||
n = min(QLINK_MAX_NR_AKM_SUITES, sme->crypto.n_akm_suites);
|
||||
aen->n_akm_suites = cpu_to_le32(n);
|
||||
|
||||
for (i = 0; i < QLINK_MAX_NR_AKM_SUITES; i++)
|
||||
for (i = 0; i < n; i++)
|
||||
aen->akm_suites[i] = cpu_to_le32(sme->crypto.akm_suites[i]);
|
||||
|
||||
aen->control_port = sme->crypto.control_port;
|
||||
|
|
|
|||
|
|
@ -903,6 +903,7 @@ struct survey_info {
|
|||
};
|
||||
|
||||
#define CFG80211_MAX_WEP_KEYS 4
|
||||
#define CFG80211_MAX_NUM_AKM_SUITES 10
|
||||
|
||||
/**
|
||||
* struct cfg80211_crypto_settings - Crypto settings
|
||||
|
|
@ -937,7 +938,7 @@ struct cfg80211_crypto_settings {
|
|||
int n_ciphers_pairwise;
|
||||
u32 ciphers_pairwise[NL80211_MAX_NR_CIPHER_SUITES];
|
||||
int n_akm_suites;
|
||||
u32 akm_suites[NL80211_MAX_NR_AKM_SUITES];
|
||||
u32 akm_suites[CFG80211_MAX_NUM_AKM_SUITES];
|
||||
bool control_port;
|
||||
__be16 control_port_ethertype;
|
||||
bool control_port_no_encrypt;
|
||||
|
|
@ -4682,6 +4683,13 @@ struct wiphy_iftype_akm_suites {
|
|||
* supported by the driver for each vif
|
||||
* @tid_config_support.peer: bitmap of attributes (configurations)
|
||||
* supported by the driver for each peer
|
||||
* @max_num_akm_suites: maximum number of AKM suites allowed for
|
||||
* configuration through %NL80211_CMD_CONNECT, %NL80211_CMD_ASSOCIATE and
|
||||
* %NL80211_CMD_START_AP. Set to NL80211_MAX_NR_AKM_SUITES if not set by
|
||||
* driver. If set by driver minimum allowed value is
|
||||
* NL80211_MAX_NR_AKM_SUITES in order to avoid compatibility issues with
|
||||
* legacy userspace and maximum allowed value is
|
||||
* CFG80211_MAX_NUM_AKM_SUITES.
|
||||
*/
|
||||
struct wiphy {
|
||||
/* assign these fields before you register the wiphy */
|
||||
|
|
@ -4833,6 +4841,8 @@ struct wiphy {
|
|||
u64 peer, vif;
|
||||
} tid_config_support;
|
||||
|
||||
u16 max_num_akm_suites;
|
||||
|
||||
char priv[0] __aligned(NETDEV_ALIGN);
|
||||
};
|
||||
|
||||
|
|
|
|||
|
|
@ -2465,6 +2465,13 @@ enum nl80211_commands {
|
|||
* @NL80211_ATTR_HE_6GHZ_CAPABILITY: HE 6 GHz Band Capability element (from
|
||||
* association request when used with NL80211_CMD_NEW_STATION).
|
||||
*
|
||||
* @NL80211_ATTR_MAX_NUM_AKM_SUITES: U16 attribute. Indicates maximum number of
|
||||
* AKM suites allowed for %NL80211_CMD_CONNECT, %NL80211_CMD_ASSOCIATE and
|
||||
* %NL80211_CMD_START_AP in %NL80211_CMD_GET_WIPHY response. If this
|
||||
* attribute is not present userspace shall consider maximum number of AKM
|
||||
* suites allowed as %NL80211_MAX_NR_AKM_SUITES which is the legacy maximum
|
||||
* number prior to the introduction of this attribute.
|
||||
*
|
||||
* @NUM_NL80211_ATTR: total number of nl80211_attrs available
|
||||
* @NL80211_ATTR_MAX: highest attribute number currently defined
|
||||
* @__NL80211_ATTR_AFTER_LAST: internal use
|
||||
|
|
@ -2942,6 +2949,8 @@ enum nl80211_attrs {
|
|||
|
||||
NL80211_ATTR_HE_6GHZ_CAPABILITY,
|
||||
|
||||
NL80211_ATTR_MAX_NUM_AKM_SUITES = 316,
|
||||
|
||||
/* add attributes here, update the policy in nl80211.c */
|
||||
|
||||
__NL80211_ATTR_AFTER_LAST,
|
||||
|
|
@ -2994,6 +3003,11 @@ enum nl80211_attrs {
|
|||
#define NL80211_HE_MIN_CAPABILITY_LEN 16
|
||||
#define NL80211_HE_MAX_CAPABILITY_LEN 54
|
||||
#define NL80211_MAX_NR_CIPHER_SUITES 5
|
||||
|
||||
/*
|
||||
* NL80211_MAX_NR_AKM_SUITES is obsolete when %NL80211_ATTR_MAX_NUM_AKM_SUITES
|
||||
* present in %NL80211_CMD_GET_WIPHY response.
|
||||
*/
|
||||
#define NL80211_MAX_NR_AKM_SUITES 2
|
||||
|
||||
#define NL80211_MIN_REMAIN_ON_CHANNEL_TIME 10
|
||||
|
|
|
|||
|
|
@ -894,6 +894,12 @@ int wiphy_register(struct wiphy *wiphy)
|
|||
return -EINVAL;
|
||||
#endif
|
||||
|
||||
if (!wiphy->max_num_akm_suites)
|
||||
wiphy->max_num_akm_suites = NL80211_MAX_NR_AKM_SUITES;
|
||||
else if (wiphy->max_num_akm_suites < NL80211_MAX_NR_AKM_SUITES ||
|
||||
wiphy->max_num_akm_suites > CFG80211_MAX_NUM_AKM_SUITES)
|
||||
return -EINVAL;
|
||||
|
||||
/* check and set up bitrates */
|
||||
ieee80211_set_bitrate_flags(wiphy);
|
||||
|
||||
|
|
|
|||
|
|
@ -656,6 +656,7 @@ const struct nla_policy nl80211_policy[NUM_NL80211_ATTR] = {
|
|||
.type = NLA_EXACT_LEN,
|
||||
.len = sizeof(struct ieee80211_he_6ghz_capa),
|
||||
},
|
||||
[NL80211_ATTR_MAX_NUM_AKM_SUITES] = { .type = NLA_REJECT },
|
||||
};
|
||||
|
||||
/* policy for the key attributes */
|
||||
|
|
@ -2574,6 +2575,10 @@ static int nl80211_send_wiphy(struct cfg80211_registered_device *rdev,
|
|||
if (nl80211_put_tid_config_support(rdev, msg))
|
||||
goto nla_put_failure;
|
||||
|
||||
if (nla_put_u16(msg, NL80211_ATTR_MAX_NUM_AKM_SUITES,
|
||||
rdev->wiphy.max_num_akm_suites))
|
||||
goto nla_put_failure;
|
||||
|
||||
/* done */
|
||||
state->split_start = 0;
|
||||
break;
|
||||
|
|
@ -9377,7 +9382,7 @@ static int nl80211_crypto_settings(struct cfg80211_registered_device *rdev,
|
|||
if (len % sizeof(u32))
|
||||
return -EINVAL;
|
||||
|
||||
if (settings->n_akm_suites > NL80211_MAX_NR_AKM_SUITES)
|
||||
if (settings->n_akm_suites > rdev->wiphy.max_num_akm_suites)
|
||||
return -EINVAL;
|
||||
|
||||
memcpy(settings->akm_suites, data, len);
|
||||
|
|
|
|||
Loading…
Reference in a new issue