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 (file)
--- 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);