DGNum/hostapd
6
0
Fork 0
Code Issues Pull requests 1 Projects Releases Packages Wiki Activity Actions

DPP: Fix GAS fragmentation for DPP Config Response from hostapd

Browse source 
The Query Response Length field was missing from GAS Initial Response
and GAS Comeback Response frames in the DPP specific code path from
hostaps GAS server. This resulted in invalid frames being used when the
DPP Config Response needed fragmentation. Fix this by adding the Query
Response Length fields into these frames.

Signed-off-by: Disha Das <dishad@codeaurora.org>
This commit is contained in:
Disha Das 2020-09-11 09:58:40 +05:30 committed by Jouni Malinen
parent de250a1984
commit 5c6c0d569a
1 changed files with 7 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
src/ap/gas_serv.c
View file

@ -1555,11 +1555,14 @@ void gas_serv_req_dpp_processing(struct hostapd_data *hapd,
di->prot = prot; di->prot = prot;
di->sd_resp = buf; di->sd_resp = buf;
di->sd_resp_pos = 0; di->sd_resp_pos = 0;
di->dpp = 1;
tx_buf = gas_build_initial_resp( tx_buf = gas_build_initial_resp(
dialog_token, WLAN_STATUS_SUCCESS, dialog_token, WLAN_STATUS_SUCCESS,
comeback_delay, 10); comeback_delay, 10 + 2);
if (tx_buf) if (tx_buf) {
gas_serv_write_dpp_adv_proto(tx_buf); gas_serv_write_dpp_adv_proto(tx_buf);
wpabuf_put_le16(tx_buf, 0);
}
} }
} else { } else {
wpa_printf(MSG_DEBUG, wpa_printf(MSG_DEBUG,
@ -1782,9 +1785,10 @@ static void gas_serv_rx_gas_comeback_req(struct hostapd_data *hapd,
tx_buf = gas_build_comeback_resp(dialog_token, tx_buf = gas_build_comeback_resp(dialog_token,
WLAN_STATUS_SUCCESS, WLAN_STATUS_SUCCESS,
dialog->sd_frag_id, more, 0, dialog->sd_frag_id, more, 0,
10 + frag_len); 10 + 2 + frag_len);
if (tx_buf) { if (tx_buf) {
gas_serv_write_dpp_adv_proto(tx_buf); gas_serv_write_dpp_adv_proto(tx_buf);
wpabuf_put_le16(tx_buf, frag_len);
wpabuf_put_buf(tx_buf, buf); wpabuf_put_buf(tx_buf, buf);
} }
} else } else