Kernel/iwd
3
0
Fork 0
mirror of https://git.kernel.org/pub/scm/network/wireless/iwd.git synced 2024-11-25 17:59:25 +01:00
Code Releases Activity

dpp: fix incorrect offchannel usage as configurator

Browse Source 
When acting as a configurator the enrollee can start on a different
channel than IWD is connected to. IWD will begin the auth process
on this channel but tell the enrollee to transition to the current
channel after the auth request. Since a configurator must be
connected (a requirement IWD enforces) we can assume a channel
transition will always be to the currently connected channel. This
allows us to simply cancel the offchannel request and wait for a
response (rather than start another offchannel).

Doing this improves the DPP performance and reduces the potential
for a lost frame during the channel transition.

This patch also addresses the comment that we should wait for the
auth request ACK before canceling the offchannel. Now a flag is
set and IWD will cancel the offchannel once the ACK is received.
This commit is contained in:
James Prestwood 2023-04-25 13:57:00 -07:00 committed by Denis Kenzior
parent 6c97ebb813
commit b080854d3e
1 changed files with 31 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

35
src/dpp.c
View File

@ -147,6 +147,7 @@ struct dpp_sm {
bool mcast_support : 1; bool mcast_support : 1;
bool roc_started : 1; bool roc_started : 1;
bool channel_switch : 1;
}; };
static bool dpp_get_started(struct l_dbus *dbus, static bool dpp_get_started(struct l_dbus *dbus,
@ -2032,6 +2033,7 @@ static void authenticate_response(struct dpp_sm *dpp, const uint8_t *from,
return; return;
} }
dpp->channel_switch = false;
dpp->current_freq = dpp->new_freq; dpp->current_freq = dpp->new_freq;
dpp_send_authenticate_confirm(dpp); dpp_send_authenticate_confirm(dpp);
@ -2112,10 +2114,14 @@ static void dpp_handle_presence_announcement(struct dpp_sm *dpp,
return; return;
/* /*
* Should we wait for an ACK then go offchannel? * Requested the peer to move to another channel for the remainder of
* the protocol. IWD's current logic prohibits a configurator from
* running while not connected, so we can assume here that the new
* frequency is the same of the connected BSS. Wait until an ACK is
* received for the auth request then cancel the offchannel request.
*/ */
if (dpp->current_freq != dpp->new_freq) if (dpp->current_freq != dpp->new_freq)
dpp_start_offchannel(dpp, dpp->new_freq); dpp->channel_switch = true;
} }
static void dpp_handle_frame(struct dpp_sm *dpp, static void dpp_handle_frame(struct dpp_sm *dpp,
@ -2211,13 +2217,34 @@ static void dpp_mlme_notify(struct l_genl_msg *msg, void *user_data)
if (dpp->state <= DPP_STATE_PRESENCE) if (dpp->state <= DPP_STATE_PRESENCE)
return; return;
if (dpp->frame_cookie != cookie)
return;
/* /*
* Only want to handle the no-ACK case. Re-transmitting an ACKed * Only want to handle the no-ACK case. Re-transmitting an ACKed
* frame likely wont do any good, at least in the case of DPP. * frame likely wont do any good, at least in the case of DPP.
*/ */
if (dpp->frame_cookie != cookie || ack) if (!ack)
return; goto retransmit;
/*
* Special handling for a channel transition when acting as a
* configurator. The auth request was sent offchannel so we need to
* wait for the ACK before going back to the connected channel.
*/
if (dpp->channel_switch) {
if (dpp->offchannel_id) {
offchannel_cancel(dpp->wdev_id, dpp->offchannel_id);
dpp->offchannel_id = 0;
}
dpp->channel_switch = false;
}
return;
retransmit:
if (dpp->frame_retry > DPP_FRAME_MAX_RETRIES) { if (dpp->frame_retry > DPP_FRAME_MAX_RETRIES) {
dpp_reset(dpp); dpp_reset(dpp);
return; return;