nl80211: Ignore netlink interface down if interface is up
The netlink event processing is delayed since they are only returned
after control returns to eloop. This can result in netlink down events
being processed at a point when the interface has is actually still
there (the first event on new interface is down) and that can result in
odd behavior especially when the resulting interface-disabled event is
delivered to AP mode interface with wpa_supplicant.
Work around this by filtering netlink down events if the matching
interface is up at the time the netlink event is processed. This fixes
an issue brought up by commit 36d84860bb
.
This commit is contained in:
parent
d28b43f66f
commit
59d249255c
3 changed files with 31 additions and 0 deletions
|
@ -601,6 +601,14 @@ static void wpa_driver_nl80211_event_rtm_newlink(void *ctx,
|
||||||
(ifi->ifi_flags & IFF_DORMANT) ? "[DORMANT]" : "");
|
(ifi->ifi_flags & IFF_DORMANT) ? "[DORMANT]" : "");
|
||||||
|
|
||||||
if (!drv->if_disabled && !(ifi->ifi_flags & IFF_UP)) {
|
if (!drv->if_disabled && !(ifi->ifi_flags & IFF_UP)) {
|
||||||
|
char namebuf[IFNAMSIZ];
|
||||||
|
if (if_indextoname(ifi->ifi_index, namebuf) &&
|
||||||
|
linux_iface_up(drv->global->ioctl_sock,
|
||||||
|
drv->first_bss.ifname) > 0) {
|
||||||
|
wpa_printf(MSG_DEBUG, "nl80211: Ignore interface down "
|
||||||
|
"event since interface %s is up", namebuf);
|
||||||
|
return;
|
||||||
|
}
|
||||||
wpa_printf(MSG_DEBUG, "nl80211: Interface down");
|
wpa_printf(MSG_DEBUG, "nl80211: Interface down");
|
||||||
if (drv->ignore_if_down_event) {
|
if (drv->ignore_if_down_event) {
|
||||||
wpa_printf(MSG_DEBUG, "nl80211: Ignore interface down "
|
wpa_printf(MSG_DEBUG, "nl80211: Ignore interface down "
|
||||||
|
|
|
@ -60,6 +60,28 @@ int linux_set_iface_flags(int sock, const char *ifname, int dev_up)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
int linux_iface_up(int sock, const char *ifname)
|
||||||
|
{
|
||||||
|
struct ifreq ifr;
|
||||||
|
int ret;
|
||||||
|
|
||||||
|
if (sock < 0)
|
||||||
|
return -1;
|
||||||
|
|
||||||
|
os_memset(&ifr, 0, sizeof(ifr));
|
||||||
|
os_strlcpy(ifr.ifr_name, ifname, IFNAMSIZ);
|
||||||
|
|
||||||
|
if (ioctl(sock, SIOCGIFFLAGS, &ifr) != 0) {
|
||||||
|
ret = errno ? -errno : -999;
|
||||||
|
wpa_printf(MSG_ERROR, "Could not read interface %s flags: %s",
|
||||||
|
ifname, strerror(errno));
|
||||||
|
return ret;
|
||||||
|
}
|
||||||
|
|
||||||
|
return !!(ifr.ifr_flags & IFF_UP);
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
int linux_get_ifhwaddr(int sock, const char *ifname, u8 *addr)
|
int linux_get_ifhwaddr(int sock, const char *ifname, u8 *addr)
|
||||||
{
|
{
|
||||||
struct ifreq ifr;
|
struct ifreq ifr;
|
||||||
|
|
|
@ -16,6 +16,7 @@
|
||||||
#define LINUX_IOCTL_H
|
#define LINUX_IOCTL_H
|
||||||
|
|
||||||
int linux_set_iface_flags(int sock, const char *ifname, int dev_up);
|
int linux_set_iface_flags(int sock, const char *ifname, int dev_up);
|
||||||
|
int linux_iface_up(int sock, const char *ifname);
|
||||||
int linux_get_ifhwaddr(int sock, const char *ifname, u8 *addr);
|
int linux_get_ifhwaddr(int sock, const char *ifname, u8 *addr);
|
||||||
int linux_set_ifhwaddr(int sock, const char *ifname, const u8 *addr);
|
int linux_set_ifhwaddr(int sock, const char *ifname, const u8 *addr);
|
||||||
int linux_br_add(int sock, const char *brname);
|
int linux_br_add(int sock, const char *brname);
|
||||||
|
|
Loading…
Reference in a new issue