From: Patrik Flykt <[email protected]>
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 files changed, 0 insertions(+), 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);
--
1.7.2.5
_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman
