summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFelix Fietkau2024-06-14 12:43:39 +0000
committerFelix Fietkau2024-06-21 09:52:25 +0000
commit3984fb0582083c5c0f511ed3a33eb17908eccb08 (patch)
tree9ee09848772436eb05fe7bc20581680605fdc4db
parent2b6772c82c6a4a498a7e5427b39ead2d60a42762 (diff)
downloadopenwrt-3984fb0582083c5c0f511ed3a33eb17908eccb08.tar.gz
hostapd: fix crash on interface setup failure
Add a missing NULL pointer check when deleting beacons Signed-off-by: Felix Fietkau <nbd@nbd.name>
-rw-r--r--package/network/services/hostapd/patches/060-nl80211-fix-crash-when-adding-an-interface-fails.patch21
1 files changed, 21 insertions, 0 deletions
diff --git a/package/network/services/hostapd/patches/060-nl80211-fix-crash-when-adding-an-interface-fails.patch b/package/network/services/hostapd/patches/060-nl80211-fix-crash-when-adding-an-interface-fails.patch
new file mode 100644
index 0000000000..1df2eff387
--- /dev/null
+++ b/package/network/services/hostapd/patches/060-nl80211-fix-crash-when-adding-an-interface-fails.patch
@@ -0,0 +1,21 @@
+From: Felix Fietkau <nbd@nbd.name>
+Date: Fri, 14 Jun 2024 14:41:16 +0200
+Subject: [PATCH] nl80211: fix crash when adding an interface fails
+
+When adding an interface fails early, the bss link is still NULL.
+Avoid crashing on deleting beacons.
+
+Signed-off-by: Felix Fietkau <nbd@nbd.name>
+---
+
+--- a/src/drivers/driver_nl80211.c
++++ b/src/drivers/driver_nl80211.c
+@@ -3071,7 +3071,7 @@ static int wpa_driver_nl80211_del_beacon
+ struct wpa_driver_nl80211_data *drv = bss->drv;
+ struct i802_link *link = nl80211_get_link(bss, link_id);
+
+- if (!link->beacon_set)
++ if (!link || !link->beacon_set)
+ return 0;
+
+ wpa_printf(MSG_DEBUG, "nl80211: Remove beacon (ifindex=%d)",