diff options
| author | Patrik Flykt <patrik.flykt@linux.intel.com> | 2012-01-25 13:49:40 +0200 |
|---|---|---|
| committer | Samuel Ortiz <sameo@linux.intel.com> | 2012-01-27 11:21:24 +0100 |
| commit | 35d3308810e424f44fb4c84d8446dd80f5f744e5 (patch) | |
| tree | 8f4b0fdfce087e30ee4b7721ea44a6db460f414b | |
| parent | 57b5f962c298199d9ff1b99a04a2fbddd98e8eeb (diff) | |
| download | connman-35d3308810e424f44fb4c84d8446dd80f5f744e5.tar.gz connman-35d3308810e424f44fb4c84d8446dd80f5f744e5.tar.bz2 connman-35d3308810e424f44fb4c84d8446dd80f5f744e5.zip | |
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.
| -rw-r--r-- | src/provider.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/provider.c b/src/provider.c index ce3382d7..a041f91e 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); |