From 9b1a20b1cf7fcfb080c353120bfc796fda56bf8a Mon Sep 17 00:00:00 2001 From: Daniel Wagner Date: Wed, 31 Oct 2012 10:33:32 +0100 Subject: [PATCH] session: Remove name variable from connman_session_bearer The string is only used when appending the bearer to the D-Bus message in append_allowed_bearers(). Let's use __connman_session_type2string() in append_allowed_bearers(). This saves a bit of memory. --- include/session.h | 1 - src/session.c | 15 ++++++++------- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/include/session.h b/include/session.h index f006e00..73f951d 100644 --- a/include/session.h +++ b/include/session.h @@ -52,7 +52,6 @@ enum connman_session_type { struct connman_session; struct connman_session_bearer { - char *name; connman_bool_t match_all; enum connman_service_type service_type; }; diff --git a/src/session.c b/src/session.c index 247ff06..d7f015a 100644 --- a/src/session.c +++ b/src/session.c @@ -377,7 +377,6 @@ static void cleanup_bearer(gpointer data) { struct connman_session_bearer *bearer = data; - g_free(bearer->name); g_free(bearer); } @@ -407,11 +406,10 @@ static int session_parse_allowed_bearers(DBusMessageIter *iter, GSList **list) return -ENOMEM; } - bearer->name = g_strdup(bearer_name); - bearer->service_type = bearer2service(bearer->name); + bearer->service_type = bearer2service(bearer_name); if (bearer->service_type == CONNMAN_SERVICE_TYPE_UNKNOWN && - g_strcmp0(bearer->name, "*") == 0) { + g_strcmp0(bearer_name, "*") == 0) { bearer->match_all = TRUE; } else { bearer->match_all = FALSE; @@ -434,7 +432,6 @@ static struct connman_session_bearer *clone_bearer( if (bearer == NULL) return NULL; - bearer->name = g_strdup(orig->name); bearer->match_all = orig->match_all; bearer->service_type = orig->service_type; @@ -518,7 +515,6 @@ GSList *connman_session_allowed_bearers_any(void) if (bearer == NULL) return NULL; - bearer->name = g_strdup(""); bearer->match_all = TRUE; bearer->service_type = CONNMAN_SERVICE_TYPE_UNKNOWN; @@ -535,9 +531,14 @@ static void append_allowed_bearers(DBusMessageIter *iter, void *user_data) for (list = info->config.allowed_bearers; list != NULL; list = list->next) { struct connman_session_bearer *bearer = list->data; + const char *name = + __connman_service_type2string(bearer->service_type); + + if (name == NULL) + name = "*"; dbus_message_iter_append_basic(iter, DBUS_TYPE_STRING, - &bearer->name); + &name); } } -- 2.7.4