From: Patrik Flykt <patrik.flykt@linux.intel.com>
Date: Wed, 25 Jan 2012 11:49:40 +0000 (+0200)
Subject: provider: Don't set provider->name when creating provider struct
X-Git-Tag: 0.79~141
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=35d3308810e424f44fb4c84d8446dd80f5f744e5;p=platform%2Fupstream%2Fconnman.git

provider: Don't set provider->name when creating provider struct

Don't unnecessarily set provider->name to the value of the provider
identifier in connman_provider_get() since it's going to be reset
immediately to the supplied provider name.
---

diff --git a/src/provider.c b/src/provider.c
index ce3382d..a041f91 100644
--- a/src/provider.c
+++ b/src/provider.c
@@ -531,8 +531,6 @@ static struct connman_provider *connman_provider_get(const char *identifier)
 
 	g_hash_table_insert(provider_hash, provider->identifier, provider);
 
-	provider->name = g_strdup(identifier);
-
 	return provider;
 }
 
@@ -613,7 +611,6 @@ int __connman_provider_create_and_connect(DBusMessage *msg)
 
 		provider->host = g_strdup(host);
 		provider->domain = g_strdup(domain);
-		g_free(provider->name);
 		provider->name = g_strdup(name);
 		provider->type = g_strdup(type);