> +++ 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