emit a single assig_vips bus message for all VIPs

src/libcharon/plugins/tnc_ifmap/tnc_ifmap_listener.c

@@ -71,9 +71,8 @@ static bool publish_device_ip_addresses(private_tnc_ifmap_listener_t *this)
  */
 static bool reload_metadata(private_tnc_ifmap_listener_t *this)
 {
-	enumerator_t *enumerator, *evips;
 	ike_sa_t *ike_sa;
-	host_t *vip;
+	enumerator_t *enumerator;
 	bool success = TRUE;
 
 	enumerator = charon->controller->create_ike_sa_enumerator(
@@ -84,21 +83,12 @@ static bool reload_metadata(private_tnc_ifmap_listener_t *this)
 		{
 			continue;
 		}
-		if (!this->ifmap->publish_ike_sa(this->ifmap, ike_sa, TRUE))
+		if (!this->ifmap->publish_ike_sa(this->ifmap, ike_sa, TRUE) ||
+			!this->ifmap->publish_virtual_ips(this->ifmap, ike_sa, TRUE))
 		{
 			success = FALSE;
 			break;
 		}
-		evips = ike_sa->create_virtual_ip_enumerator(ike_sa, FALSE);
-		while (evips->enumerate(evips, &vip))
-		{
-			if (!this->ifmap->publish_virtual_ip(this->ifmap, ike_sa, vip, TRUE))
-			{
-				success = FALSE;
-				break;
-			}
-		}
-		evips->destroy(evips);
 	}
 	enumerator->destroy(enumerator);
 
@@ -115,11 +105,10 @@ METHOD(listener_t, ike_updown, bool,
 	return TRUE;
 }
 
-METHOD(listener_t, assign_vip, bool,
-	private_tnc_ifmap_listener_t *this, ike_sa_t *ike_sa, host_t *vip,
-	bool assign)
+METHOD(listener_t, assign_vips, bool,
+	private_tnc_ifmap_listener_t *this, ike_sa_t *ike_sa, bool assign)
 {
-	this->ifmap->publish_virtual_ip(this->ifmap, ike_sa, vip, assign);
+	this->ifmap->publish_virtual_ips(this->ifmap, ike_sa, assign);
 	return TRUE;
 }
 
@@ -163,7 +152,7 @@ tnc_ifmap_listener_t *tnc_ifmap_listener_create(bool reload)
 		.public = {
 			.listener = {
 				.ike_updown = _ike_updown,
-				.assign_vip = _assign_vip,
+				.assign_vips = _assign_vips,
 				.alert = _alert,
 			},
 			.destroy = _destroy,

src/libcharon/plugins/tnc_ifmap/tnc_ifmap_soap.c

@@ -579,12 +579,14 @@ METHOD(tnc_ifmap_soap_t, publish_device_ip, bool,
 	return success;
 }
 
-METHOD(tnc_ifmap_soap_t, publish_virtual_ip, bool,
-	private_tnc_ifmap_soap_t *this, ike_sa_t *ike_sa, host_t *vip, bool assign)
+METHOD(tnc_ifmap_soap_t, publish_virtual_ips, bool,
+	private_tnc_ifmap_soap_t *this, ike_sa_t *ike_sa, bool assign)
 {
 	tnc_ifmap_soap_msg_t *soap_msg;
 	xmlNodePtr request, node;
 	u_int32_t ike_sa_id;
+	enumerator_t *enumerator;
+	host_t *vip;
 	bool success;
 
 	/* extract relevant data from IKE_SA*/
@@ -593,26 +595,31 @@ METHOD(tnc_ifmap_soap_t, publish_virtual_ip, bool,
 	/* build publish request */
 	request = create_publish_request(this);
 
-	/**
-	 * update or delete access-request-ip metadata for a virtual IP address
-	 */
-	if (assign)
+	enumerator = ike_sa->create_virtual_ip_enumerator(ike_sa, FALSE);
+	while (enumerator->enumerate(enumerator, &vip))
 	{
-		node = xmlNewNode(NULL, "update");
-	}
-	else
-	{
-		node = create_delete_filter(this, "access-request-ip");
-	}
-	xmlAddChild(request, node);
+		/**
+		 * update or delete access-request-ip metadata for a virtual IP address
+		 */
+		if (assign)
+		{
+			node = xmlNewNode(NULL, "update");
+		}
+		else
+		{
+			node = create_delete_filter(this, "access-request-ip");
+		}
+		xmlAddChild(request, node);
 
-	/* add access-request, virtual ip-address and [if assign] metadata */
-	xmlAddChild(node, create_access_request(this, ike_sa_id));
-	xmlAddChild(node, create_ip_address(this, vip));
-	if (assign)
-	{
-		xmlAddChild(node, create_metadata(this, "access-request-ip"));
+		/* add access-request, virtual ip-address and [if assign] metadata */
+			xmlAddChild(node, create_access_request(this, ike_sa_id));
+			xmlAddChild(node, create_ip_address(this, vip));
+			if (assign)
+		{
+			xmlAddChild(node, create_metadata(this, "access-request-ip"));
+		}
 	}
+	enumerator->destroy(enumerator);
 
 	soap_msg = tnc_ifmap_soap_msg_create(this->uri, this->user_pass, this->tls);
 	success = soap_msg->post(soap_msg, request, "publishReceived", NULL);
@@ -893,7 +900,7 @@ tnc_ifmap_soap_t *tnc_ifmap_soap_create()
 			.purgePublisher = _purgePublisher,
 			.publish_ike_sa = _publish_ike_sa,
 			.publish_device_ip = _publish_device_ip,
-			.publish_virtual_ip = _publish_virtual_ip,
+			.publish_virtual_ips = _publish_virtual_ips,
 			.publish_enforcement_report = _publish_enforcement_report,
 			.endSession = _endSession,
 			.get_session_id = _get_session_id,

src/libcharon/plugins/tnc_ifmap/tnc_ifmap_soap.h

@@ -71,15 +71,14 @@ struct tnc_ifmap_soap_t {
 	bool (*publish_device_ip)(tnc_ifmap_soap_t *this, host_t *host);
 
 	/**
-	 * Publish Virtual IP  access-request-ip metadata
+	 * Publish Virtual IP access-request-ip metadata
 	 *
-	 * @param ike_sa		IKE_SA for which metadata is published
-	 * @param vip			Virtual IP address of peer
+	 * @param ike_sa		IKE_SA for which Virtual IP metadata is published
 	 * @param assign		TRUE if assigned, FALSE if removed
 	 * @return				TRUE if command was successful
 	 */
-	bool (*publish_virtual_ip)(tnc_ifmap_soap_t *this, ike_sa_t *ike_sa,
-							   host_t *vip, bool assign);
+	bool (*publish_virtual_ips)(tnc_ifmap_soap_t *this, ike_sa_t *ike_sa,
+								bool assign);
 
 	/**
 	 * Publish enforcement-report metadata