This is an automated email from the ASF dual-hosted git repository. casaroli pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git
commit 225333b5da33163b3b9cfb11dbc5ca4d88347493 Author: yangjianqing <[email protected]> AuthorDate: Mon Dec 1 09:42:44 2025 +0800 qemu/wifi:Fixed a risky issue with dereferencing null pointers in Coverity. Fixed the null pointer dereference issue. Signed-off-by: yangjianqing <[email protected]> --- drivers/net/wifi_sim.c | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/drivers/net/wifi_sim.c b/drivers/net/wifi_sim.c index a0043f4e3fd..4c4d0d0b098 100644 --- a/drivers/net/wifi_sim.c +++ b/drivers/net/wifi_sim.c @@ -2004,15 +2004,18 @@ int wifi_sim_init(FAR struct wifi_sim_lowerhalf_s *netdev) void wifi_sim_remove(FAR struct wifi_sim_lowerhalf_s *netdev) { - FAR struct wifi_sim_s *sta = (FAR struct wifi_sim_s *)netdev->wifi; + FAR struct wifi_sim_s *wifidev = (FAR struct wifi_sim_s *)netdev->wifi; - if (sta && sta->state == WLAN_STA_STATE_CONNECTED) + if (wifidev) { - wifidriver_start_disconnect(sta); - } + if (wifidev->state == WLAN_STA_STATE_CONNECTED) + { + wifidriver_start_disconnect(wifidev); + } - nxmutex_destroy(&sta->ap_lock); - kmm_free(netdev->wifi); + nxmutex_destroy(&wifidev->ap_lock); + kmm_free(wifidev); + } } /****************************************************************************
