From b9ee515dc910f84cc97fc21486172b2e6b3ed217 Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Tue, 19 Jan 2016 22:05:09 +0100 Subject: [PATCH] supplicant: remove check for iface instance after nm_supplicant_interface_new() A plain constructor for a GObject cannot fail. (cherry picked from commit a8165611a844514985f85767170b946edc0cfb65) --- src/supplicant-manager/nm-supplicant-manager.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/supplicant-manager/nm-supplicant-manager.c b/src/supplicant-manager/nm-supplicant-manager.c index c46b1974c3..d15aa166bc 100644 --- a/src/supplicant-manager/nm-supplicant-manager.c +++ b/src/supplicant-manager/nm-supplicant-manager.c @@ -87,11 +87,9 @@ nm_supplicant_manager_iface_get (NMSupplicantManager * self, priv->fast_supported, priv->ap_support, start_now); - if (iface) { - g_hash_table_insert (priv->ifaces, - (char *) nm_supplicant_interface_get_ifname (iface), - iface); - } + g_hash_table_insert (priv->ifaces, + (char *) nm_supplicant_interface_get_ifname (iface), + iface); } else { /* nm_supplicant_manager_iface_get() and release() implements no form of ref-counting * to properly handle reusing a cached instance. It's also unclear whether that is