AP MLD: Deinitialize BSS link on interface setup failure

Fix a segmentation fault in hostapd when interface setup fails in
hostapd_add_iface() due to invalid channel information. In this case,
the driver is deinitialized but the BSS link is not, leading to a crash
when the interface is brought down and up during stability testing.

Deinitialize the BSS link if interface setup fails to ensure proper
cleanup and prevent the crash.

Signed-off-by: Ramasamy Kaliappan <ramasamy.kaliappan@oss.qualcomm.com>
Signed-off-by: Aaradhana Sahu <aaradhana.sahu@oss.qualcomm.com>

diff --git a/src/ap/hostapd.c b/src/ap/hostapd.c
index 8da0ccafe07d9f91c4679e528b29232997492f41..161049fc3a3611962d111ce77df1f494051a9b34 100644 (file)
--- a/src/ap/hostapd.c
+++ b/src/ap/hostapd.c
@@ -3879,6 +3879,8 @@ int hostapd_add_iface(struct hapd_interfaces *interfaces, char *buf)
                        }
 
                        if (hostapd_setup_interface(hapd_iface)) {
+                               /* Deinit link for the first bss */
+                               hostapd_bss_link_deinit(hapd_iface->bss[0]);
                                hostapd_deinit_driver(
                                        hapd_iface->bss[0]->driver,
                                        hapd_iface->bss[0]->drv_priv,