From 98e33605b4d8e8c2663a691b45b2649471f3aefa 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 6abb0e0602c0..f4dc09655b9c 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, ETS_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_IEEE80211KV)