From: Jouni Malinen <j@w1.fi>
Date: Sun, 17 Oct 2010 18:36:04 +0000 (+0300)
Subject: WPS: Do not drop subscriptions based on max queue length
X-Git-Tag: hostap-1-bp~957
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=99ba7f889b95dfc8e895a0d691238260124a414d;p=thirdparty%2Fhostap.git

WPS: Do not drop subscriptions based on max queue length

UPnP event subscriptions are not supposed to be dropped based on
whether events can be delivered quickly enough. Leave dropping to
happen only based on failed deliveries to avoid issues with a burst
of events kicking out still active subscribers.
---

diff --git a/src/wps/wps_upnp.c b/src/wps/wps_upnp.c
index d2b83155d..97eec1d52 100644
--- a/src/wps/wps_upnp.c
+++ b/src/wps/wps_upnp.c
@@ -527,13 +527,8 @@ static void upnp_wps_device_send_event(struct upnp_wps_device_sm *sm)
 
 	dl_list_for_each_safe(s, tmp, &sm->subscriptions, struct subscription,
 			      list) {
-		if (event_add(s, buf, sm->wlanevent_type ==
-			      UPNP_WPS_WLANEVENT_TYPE_PROBE) == 1) {
-			wpa_printf(MSG_INFO, "WPS UPnP: Dropping "
-				   "subscriber %p due to event backlog", s);
-			dl_list_del(&s->list);
-			subscription_destroy(s);
-		}
+		event_add(s, buf,
+			  sm->wlanevent_type == UPNP_WPS_WLANEVENT_TYPE_PROBE);
 	}
 
 	wpabuf_free(buf);