On Mon, Apr 19, 2021 at 11:05:49PM +0000, Andrew Beltrano wrote: > Add 10s timeout for receipt of Configuration Request frame from > enrollee. > --- This needs a Signed-off-by: line or well, like I noted for 1/2, I'd actually merge this into the same patch with the more complete commit message. > diff --git a/wpa_supplicant/dpp_supplicant.c b/wpa_supplicant/dpp_supplicant.c > @@ -2968,6 +2992,8 @@ wpas_dpp_gas_req_handler(void *ctx, void *resp_ctx, const u8 *sa, > + if (auth->configurator) > + eloop_cancel_timeout(wpas_dpp_conf_req_rx_wait_timeout, wpa_s, NULL); Similarly to the hostapd case, this needs to be canceled in more places. At least wpa_dpp_auth_init() and wpas_dpp_deinit() need this for the same reasons as the hostapd patch. -- Jouni Malinen PGP id EFC895FA _______________________________________________ Hostap mailing list Hostap@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/hostap
