Update to network-manager-applet 1.2-beta3

This commit is contained in:
Lubomir Rintel 2016-03-29 17:51:29 +02:00
parent fd991c7764
commit 3e8ad6f754
4 changed files with 8 additions and 52 deletions

1
.gitignore vendored
View File

@ -26,3 +26,4 @@
/network-manager-applet-1.2.0.tar.xz
/network-manager-applet-1.1.90.tar.xz
/network-manager-applet-1.1.91.tar.xz
/network-manager-applet-1.1.92.tar.xz

View File

@ -1,46 +0,0 @@
From 22468c05291d3d88ddc68f8983bffe54f29f5f82 Mon Sep 17 00:00:00 2001
From: Thomas Haller <thaller@redhat.com>
Date: Fri, 4 Mar 2016 11:03:57 +0100
Subject: [PATCH] applet: workaround crash in applet code
rh#1313866 and rh#1314650 show a crash due to "g_assert (s_con)".
It's unclear how this can happen, but for now downgrade the crash
to a warning.
https://bugzilla.redhat.com/show_bug.cgi?id=1313866
https://bugzilla.redhat.com/show_bug.cgi?id=1314650
---
src/applet.c | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/src/applet.c b/src/applet.c
index 7ba7744..85f535c 100644
--- a/src/applet.c
+++ b/src/applet.c
@@ -1082,7 +1082,6 @@ 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,10 +1089,12 @@ applet_get_first_active_vpn_connection (NMApplet *applet,
if (!connection)
continue;
- s_con = nm_connection_get_setting_connection (connection);
- g_assert (s_con);
+ /* 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)));
- if (!strcmp (nm_setting_connection_get_connection_type (s_con), NM_SETTING_VPN_SETTING_NAME)) {
+ if (nm_streq0 (nm_connection_get_connection_type (connection), NM_SETTING_VPN_SETTING_NAME)) {
if (out_state)
*out_state = nm_vpn_connection_get_vpn_state (NM_VPN_CONNECTION (candidate));
return candidate;
--
2.5.0

View File

@ -3,7 +3,7 @@
%global nm_version 1:1.1.0
%global obsoletes_ver 1:0.9.7
%global snapshot .beta2
%global snapshot .beta3
Name: network-manager-applet
Summary: A network control and status applet for NetworkManager
@ -14,9 +14,8 @@ License: GPLv2+
URL: http://www.gnome.org/projects/NetworkManager/
Obsoletes: NetworkManager-gnome < %{obsoletes_ver}
Source: https://download.gnome.org/sources/network-manager-applet/1.1/%{name}-1.1.91.tar.xz
Source: https://download.gnome.org/sources/network-manager-applet/1.1/%{name}-1.1.92.tar.xz
Patch0: nm-applet-no-notifications.patch
Patch1: 0001-applet-workaround-crash-in-applet-code.patch
Requires: NetworkManager >= %{nm_version}
Requires: NetworkManager-glib >= %{nm_version}
@ -113,9 +112,8 @@ nm-applet, nm-connection-editor, and the GNOME control center.
This package deprecates libnm-gtk.
%prep
%setup -q -n %{name}-1.1.91
%setup -q -n %{name}-1.1.92
%patch0 -p1
%patch1 -p1
%build
autoreconf -i -f
@ -247,6 +245,9 @@ glib-compile-schemas %{_datadir}/glib-2.0/schemas &>/dev/null || :
%changelog
* Tue Mar 29 2016 Lubomir Rintel <lkundrak@v3.sk> - 1.2.0-0.3.beta3
- Update to network-manager-applet 1.2-beta3
* Mon Mar 07 2016 Lubomir Rintel <lkundrak@v3.sk> - 1.2.0-0.3.beta2
- Update to network-manager-applet 1.2-beta2

View File

@ -1 +1 @@
a7f7953e92ad0a868920dda54e67a1fc network-manager-applet-1.1.91.tar.xz
f9ac287bfc91c846dd2b1ff53ab50b99 network-manager-applet-1.1.92.tar.xz