From 3c2166d63c3f8db9699bd29b152121ca63c70415 Mon Sep 17 00:00:00 2001 From: Jouni Malinen Date: Sat, 21 Nov 2009 22:00:33 +0200 Subject: [PATCH] WPS: Do not try to send byebye advertisements if socket is not valid If initialization fails, we could potentially try to sendto() on -1 socket which would fail. No point in doing that, so just return early from the function. --- src/wps/wps_upnp_ssdp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/wps/wps_upnp_ssdp.c b/src/wps/wps_upnp_ssdp.c index 93706ea47..d48bfdb2f 100644 --- a/src/wps/wps_upnp_ssdp.c +++ b/src/wps/wps_upnp_ssdp.c @@ -239,7 +239,7 @@ void advertisement_state_machine_stop(struct upnp_wps_device_sm *sm, struct sockaddr_in dest; eloop_cancel_timeout(advertisement_state_machine_handler, NULL, sm); - if (!send_byebye) + if (!send_byebye || sm->multicast_sd < 0) return; a->type = ADVERTISE_DOWN;