From efe5cf79c0eb61b86958f5cc58c7361121d0eede Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Thu, 29 Mar 2018 11:09:11 +0200 Subject: [PATCH] core: simplify NMActiveConnection.get_property to not create temporary GPtrArrray Fixes: c050fb7cd2160f0b74ba8a0760e717e3fe329066 --- src/nm-active-connection.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/nm-active-connection.c b/src/nm-active-connection.c index a24e946d31..33da0aa131 100644 --- a/src/nm-active-connection.c +++ b/src/nm-active-connection.c @@ -1172,7 +1172,7 @@ get_property (GObject *object, guint prop_id, GValue *value, GParamSpec *pspec) { NMActiveConnectionPrivate *priv = NM_ACTIVE_CONNECTION_GET_PRIVATE ((NMActiveConnection *) object); - GPtrArray *devices; + char **strv; NMDevice *master_device = NULL; switch (prop_id) { @@ -1202,11 +1202,10 @@ get_property (GObject *object, guint prop_id, g_value_set_string (value, priv->specific_object ? priv->specific_object : "/"); break; case PROP_DEVICES: - devices = g_ptr_array_sized_new (2); + strv = g_new0 (char *, 2); if (priv->device && priv->state < NM_ACTIVE_CONNECTION_STATE_DEACTIVATED) - g_ptr_array_add (devices, g_strdup (nm_dbus_object_get_path (NM_DBUS_OBJECT (priv->device)))); - g_ptr_array_add (devices, NULL); - g_value_take_boxed (value, (char **) g_ptr_array_free (devices, FALSE)); + strv[0] = g_strdup (nm_dbus_object_get_path (NM_DBUS_OBJECT (priv->device))); + g_value_take_boxed (value, strv); break; case PROP_STATE: if (priv->state_set)