From: Jukka Rissanen <jukka.rissanen@linux.intel.com>
Date: Tue, 19 Feb 2013 08:45:04 +0000 (+0200)
Subject: vpn-provider: Do not send state property if state is not changed
X-Git-Tag: 1.12~66
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=ce3ebabc5b922c883e17b775211dfdfc3430d527;p=platform%2Fupstream%2Fconnman.git

vpn-provider: Do not send state property if state is not changed
---

diff --git a/vpn/vpn-provider.c b/vpn/vpn-provider.c
index 3cd7c6d..6a77c1e 100644
--- a/vpn/vpn-provider.c
+++ b/vpn/vpn-provider.c
@@ -1191,12 +1191,14 @@ static int provider_indicate_state(struct vpn_provider *provider,
 				enum vpn_provider_state state)
 {
 	const char *str;
+	enum vpn_provider_state old_state;
 
 	str = state2string(state);
 	DBG("provider %p state %s/%d", provider, str, state);
 	if (str == NULL)
 		return -EINVAL;
 
+	old_state = provider->state;
 	provider->state = state;
 
 	if (state == VPN_PROVIDER_STATE_READY) {
@@ -1214,9 +1216,11 @@ static int provider_indicate_state(struct vpn_provider *provider,
 					append_ipv6, provider);
 	}
 
-	connman_dbus_property_changed_basic(provider->path,
+	if (old_state != state)
+		connman_dbus_property_changed_basic(provider->path,
 					VPN_CONNECTION_INTERFACE, "State",
 					DBUS_TYPE_STRING, &str);
+
 	/*
 	 * We do not stay in failure state as clients like connmand can
 	 * get confused about our current state.