diff mbox series

[1/1] cfg80211: Allow P2P client interface to indicate successful 4way handshake by port authorization

Message ID ef25cb49fcb921df2e5d99e574f65e8a009cc52c.1655905440.git.vinayak.yadawad@broadcom.com (mailing list archive)
State Accepted
Delegated to: Johannes Berg
Headers show
Series [1/1] cfg80211: Allow P2P client interface to indicate successful 4way handshake by port authorization | expand

Commit Message

Vinayak Yadawad June 22, 2022, 1:47 p.m. UTC
In case of 4way handshake offload, cfg80211_port_authorized
enables driver to indicate successful 4way handshake to cfg80211 layer.
Currently this path of port authorization is restricted to
interface type NL80211_IFTYPE_STATION. This patch extends
the use of port authorization API for P2P client as well.

Signed-off-by: Vinayak Yadawad <vinayak.yadawad@broadcom.com>
---
 net/wireless/sme.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Kalle Valo June 29, 2022, 5:23 p.m. UTC | #1
Vinayak Yadawad <vinayak.yadawad@broadcom.com> writes:

> This electronic communication and the information and any files transmitted 
> with it, or attached to it, are confidential and are intended solely for 

These kind of disclaimers do not belong to public lists.
Kalle Valo June 30, 2022, 6:32 a.m. UTC | #2
Vinayak Yadawad <vinayak.yadawad@broadcom.com> writes:

>>>These kind of disclaimers do not belong to public lists.  
> This disclosure got added by mistake, I will resend the patch again if needed without the disclaimer footer.

Also please don't top post or use HTML in emails, it's best to ditch
Outlook when sending emails to upstream lists. More information in the
wiki link below.
Florian Fainelli June 30, 2022, 6:19 p.m. UTC | #3
On 6/29/22 23:32, Kalle Valo wrote:
> Vinayak Yadawad <vinayak.yadawad@broadcom.com> writes:
> 
>>>> These kind of disclaimers do not belong to public lists.
>> This disclosure got added by mistake, I will resend the patch again if needed without the disclaimer footer.
> 
> Also please don't top post or use HTML in emails, it's best to ditch
> Outlook when sending emails to upstream lists. More information in the
> wiki link below.
> 

This submission ought to be more correct, even if it somehow ended up 
with the same Message-Id:

https://lore.kernel.org/linux-wireless/ef25cb49fcb921df2e5d99e574f65e8a009cc52c.1655905440.git.vinayak.yadawad@broadcom.com/#r
diff mbox series

Patch

diff --git a/net/wireless/sme.c b/net/wireless/sme.c
index ff4d48fcbfb2..607a68911047 100644
--- a/net/wireless/sme.c
+++ b/net/wireless/sme.c
@@ -1031,7 +1031,8 @@  void __cfg80211_port_authorized(struct wireless_dev *wdev, const u8 *bssid)
 {
 	ASSERT_WDEV_LOCK(wdev);
 
-	if (WARN_ON(wdev->iftype != NL80211_IFTYPE_STATION))
+	if (WARN_ON(wdev->iftype != NL80211_IFTYPE_STATION &&
+		    wdev->iftype != NL80211_IFTYPE_P2P_CLIENT))
 		return;
 
 	if (WARN_ON(!wdev->current_bss) ||