From 35d3308810e424f44fb4c84d8446dd80f5f744e5 Mon Sep 17 00:00:00 2001 From: Patrik Flykt Date: Wed, 25 Jan 2012 13:49:40 +0200 Subject: [PATCH] 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. --- src/provider.c | 3 --- 1 file changed, 3 deletions(-) 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); -- 2.7.4