From 6c14120c7f94e53f0f7869f06ac2ae6250c245b9 Mon Sep 17 00:00:00 2001 From: jgujarathi Date: Tue, 16 May 2023 13:49:59 +0530 Subject: [PATCH] wpa_supplicant : Fix occasional crash during btm roam. Fix crash due to premature deallocation of neighbour report elements by scan_done. Post event to btm_rrm_t task to handle scan_done serially rather than when btm_rrm_t task is processing btm request rx frame. --- components/wpa_supplicant/esp_supplicant/src/esp_scan.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/components/wpa_supplicant/esp_supplicant/src/esp_scan.c b/components/wpa_supplicant/esp_supplicant/src/esp_scan.c index 443ebbcd47..be93dc71af 100644 --- a/components/wpa_supplicant/esp_supplicant/src/esp_scan.c +++ b/components/wpa_supplicant/esp_supplicant/src/esp_scan.c @@ -21,6 +21,7 @@ #include "esp_scan_i.h" #include "common/wnm_sta.h" #include "esp_scan_i.h" +#include "esp_common_i.h" extern struct wpa_supplicant g_wpa_supp; @@ -34,7 +35,13 @@ static void scan_done_event_handler(void *arg, STATUS status) wpa_s->type &= ~(1 << WLAN_FC_STYPE_BEACON) & ~(1 << WLAN_FC_STYPE_PROBE_RESP); esp_wifi_register_mgmt_frame_internal(wpa_s->type, wpa_s->subtype); } - esp_supplicant_handle_scan_done_evt(); +#ifdef CONFIG_SUPPLICANT_TASK + if (esp_supplicant_post_evt(SIG_SUPPLICANT_SCAN_DONE, 0) != 0) { + wpa_printf(MSG_ERROR, "Posting of scan done failed!"); + } +#else + esp_supplicant_handle_scan_done_evt(); +#endif /*CONFIG_SUPPLICANT_TASK*/ } #if defined(CONFIG_WPA_11KV_SUPPORT)