[PATCH] P2Ps: When peer has stopped sending adv_service_instance then we should clear existing dev->info.p2ps_instance.

Nishant Chaprana n.chaprana at samsung.com
Tue Jul 5 02:41:45 PDT 2016


This patch fixes below scenario:
When peer device stops sending adv_service_instance, supplicant does not remove
old dev->info.p2ps_instance from device's property.
This variable should be updated as per peer behaviour and should cleared when
peer stops sending this information.

Signed-off-by: Nishant Chaprana <n.chaprana at samsung.com>
---
 src/p2p/p2p.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/p2p/p2p.c b/src/p2p/p2p.c
index 47abe21..8fa447f 100644
--- a/src/p2p/p2p.c
+++ b/src/p2p/p2p.c
@@ -788,11 +788,11 @@ int p2p_add_device(struct p2p_data *p2p, const u8 *addr, int freq,
 		dev->oper_ssid_len = msg.ssid[1];
 	}
 
-	if (msg.adv_service_instance && msg.adv_service_instance_len) {
-		wpabuf_free(dev->info.p2ps_instance);
+	wpabuf_free(dev->info.p2ps_instance);
+	dev->info.p2ps_instance = NULL;
+	if (msg.adv_service_instance && msg.adv_service_instance_len)
 		dev->info.p2ps_instance = wpabuf_alloc_copy(
 			msg.adv_service_instance, msg.adv_service_instance_len);
-	}
 
 	if (freq >= 2412 && freq <= 2484 && msg.ds_params &&
 	    *msg.ds_params >= 1 && *msg.ds_params <= 14) {
-- 
1.9.1




More information about the Hostap mailing list