Re: [PATCH v2] cfg80211: Include the PMK and PMKID in NL80211_CMD_EXTERNAL_AUTH

[Johannes Berg]

> +++ b/net/wireless/nl80211.c
> @@ -13098,6 +13098,12 @@ static int nl80211_external_auth(struct sk_buff 
> *skb, struct genl_info *info)
>       if (!info->attrs[NL80211_ATTR_STATUS_CODE])
>               return -EINVAL;
>  
> +     if ((info->attrs[NL80211_ATTR_PMK] &&
> +          !info->attrs[NL80211_ATTR_PMKID]) ||
> +         (info->attrs[NL80211_ATTR_PMKID] &&
> +          !info->attrs[NL80211_ATTR_PMK]))
> +             return -EINVAL;

This constitutes a netlink API change, so no, can't be right? PMKID was
perfectly reasonable to pass by itself before.

johannes