[network-manager-applet] Revert "applet: workaround crash in applet code"
- From: Lubomir Rintel <lkundrak src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [network-manager-applet] Revert "applet: workaround crash in applet code"
- Date: Tue, 29 Mar 2016 14:08:47 +0000 (UTC)
commit c86d66632ad76aa4a93593a07a9b4e25f18efbf7
Author: Lubomir Rintel <lkundrak v3 sk>
Date: Tue Mar 29 15:40:54 2016 +0200
Revert "applet: workaround crash in applet code"
Fixed in libnm.
This reverts commit 22468c05291d3d88ddc68f8983bffe54f29f5f82.
src/applet.c | 9 ++++-----
1 files changed, 4 insertions(+), 5 deletions(-)
---
diff --git a/src/applet.c b/src/applet.c
index c423f49..0fe14e7 100644
--- a/src/applet.c
+++ b/src/applet.c
@@ -1083,6 +1083,7 @@ applet_get_first_active_vpn_connection (NMApplet *applet,
for (i = 0; active_list && (i < active_list->len); i++) {
NMActiveConnection *candidate;
NMConnection *connection;
+ NMSettingConnection *s_con;
candidate = g_ptr_array_index (active_list, i);
@@ -1090,12 +1091,10 @@ applet_get_first_active_vpn_connection (NMApplet *applet,
if (!connection)
continue;
- /* FIXME: rh#1313866 crashed on the following invariants for unknown reasons.
- * Downgrade crash to warning. */
- g_warn_if_fail (NM_IS_CONNECTION (connection));
- g_warn_if_fail (NM_IS_SETTING (nm_connection_get_setting_connection (connection)));
+ s_con = nm_connection_get_setting_connection (connection);
+ g_assert (s_con);
- if (nm_streq0 (nm_connection_get_connection_type (connection), NM_SETTING_VPN_SETTING_NAME)) {
+ if (!strcmp (nm_setting_connection_get_connection_type (s_con), NM_SETTING_VPN_SETTING_NAME))
{
if (out_state)
*out_state = nm_vpn_connection_get_vpn_state (NM_VPN_CONNECTION (candidate));
return candidate;
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]