On 28/09/2021 12:20, Andre Heider wrote:
fyi brport/proxyarp was still "0" too, which probably means that the
netidf condition "if (up && ifname != vif->ifname)" doesn't eval to true
as that's the only spot setting "dev->wireless_proxyarp".
That pointed me in the right direction, attached netifd patch fixes the
issue for me. lol.From 76b389cde9de3208936c8cc4ed35cf42992a8dc7 Mon Sep 17 00:00:00 2001
From: Andre Heider <a.hei...@gmail.com>
Date: Tue, 28 Sep 2021 13:29:27 +0200
Subject: [PATCH] wireless: fix applying wireless devices attributes on hotplug
events
Hotplug events pass their own 'ifname' copy, so we need to compare the
strings, not just the pointers.
Signed-off-by: Andre Heider <a.hei...@gmail.com>
---
wireless.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/wireless.c b/wireless.c
index fbd42ed..4b1fb8e 100644
--- a/wireless.c
+++ b/wireless.c
@@ -328,7 +328,7 @@ static void wireless_interface_handle_link(struct wireless_interface *vif, const
if (!ifname)
ifname = vif->ifname;
- if (up && ifname != vif->ifname) {
+ if (up && !strcmp(ifname, vif->ifname)) {
struct device *dev = device_get(ifname, 2);
if (dev) {
dev->wireless_isolate = vif->isolate;
--
2.33.0
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel
