From 5a0909caaff498d61b5cf17162fd0c5c5b381b8b Mon Sep 17 00:00:00 2001 From: Hans Dedecker Date: Thu, 12 Feb 2015 17:41:15 +0100 Subject: [PATCH 1/1] netifd: Fix possible hotplug race conditions Don't drop ifup/ifdown events in case an interface event is cached as it leads to possible race conditions (eg firewall not being reloaded as ifup is dropped) when multiple interface events are fired in a short timeframe (eg multiple PPP link flaps). Always overwrite the cached interface event except for the interface update event so the hotplug scripts are launched with the last known status. Signed-off-by: Hans Dedecker --- interface-event.c | 19 ++++--------------- 1 file changed, 4 insertions(+), 15 deletions(-) diff --git a/interface-event.c b/interface-event.c index 24af8f5..cfbc15c 100644 --- a/interface-event.c +++ b/interface-event.c @@ -120,22 +120,11 @@ interface_queue_event(struct interface *iface, enum interface_event ev) if (current == iface) { /* an event for iface is being processed */ if (!list_empty(&iface->hotplug_list)) { - /* an additional event for iface is pending */ - if ((ev != current_ev || ev == IFEV_UPDATE) && - !(iface->hotplug_ev == IFEV_UP && ev == IFEV_UPDATE)) { - /* if incoming event is different from the one - * being handled or if it is an update, - * overwrite pending event, but never - * overwrite an ifup with an ifupdate */ + /* an additional event for iface is pending */ + /* overwrite pending event if it differs from */ + /* an update */ + if (ev != IFEV_UPDATE) iface->hotplug_ev = ev; - } - else if (ev == current_ev && ev != IFEV_UPDATE) { - /* if incoming event is not an ifupdate - * and is the same as the one that is - * being handled, remove it from the - * pending list */ - list_del_init(&iface->hotplug_list); - } } else { /* no additional event for iface is pending */ -- 2.11.0