Import from AlmaLinux stable repository
This commit is contained in:
parent
fd786b9f95
commit
3f1fc1579d
@ -1 +0,0 @@
|
|||||||
a3ec22a8e76f3358d9f69dc505d22267e936dbae SOURCES/NetworkManager-libreswan-1.2.10.tar.xz
|
|
@ -0,0 +1,68 @@
|
|||||||
|
From 4be4c56b4f8a52b1cd5f8aadee273706c28ae332 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Beniamino Galvani <bgalvani@redhat.com>
|
||||||
|
Date: Sat, 13 Jan 2024 18:10:02 +0100
|
||||||
|
Subject: [PATCH 1/1] service: fix wrong refcounting in D-Bus handler for
|
||||||
|
Callback()
|
||||||
|
|
||||||
|
The Callback() D-Bus method is handled via a GDBus-generated skeleton
|
||||||
|
code in nm-libreswan-helper-service-dbus.c, function
|
||||||
|
_nmdbus_libreswan_helper_skeleton_handle_method_call(). The function
|
||||||
|
emits signal "handle-callback" to let the program handle the incoming
|
||||||
|
method. As documented in the GDoc comments, the signal handler must
|
||||||
|
return TRUE if it handles the call.
|
||||||
|
|
||||||
|
```
|
||||||
|
/**
|
||||||
|
* NMDBusLibreswanHelper::handle-callback:
|
||||||
|
* @object: A #NMDBusLibreswanHelper.
|
||||||
|
* @invocation: A #GDBusMethodInvocation.
|
||||||
|
* @arg_environment: Argument passed by remote caller.
|
||||||
|
|
||||||
|
* Signal emitted when a remote caller is invoking the Callback()
|
||||||
|
D-Bus method.
|
||||||
|
|
||||||
|
* If a signal handler returns %TRUE, it means the signal handler
|
||||||
|
will handle the invocation (e.g. take a reference to @invocation
|
||||||
|
and eventually call nmdbus_libreswan_helper_complete_callback()
|
||||||
|
or e.g. g_dbus_method_invocation_return_error() on it) and no
|
||||||
|
other signal handlers will run. If no signal handler handles the
|
||||||
|
invocation, the %G_DBUS_ERROR_UNKNOWN_METHOD error is returned.
|
||||||
|
|
||||||
|
* Returns: %G_DBUS_METHOD_INVOCATION_HANDLED or %TRUE if the
|
||||||
|
invocation was handled, %G_DBUS_METHOD_INVOCATION_UNHANDLED or
|
||||||
|
%FALSE to let other signal handlers run.
|
||||||
|
*/
|
||||||
|
```
|
||||||
|
|
||||||
|
At the moment, in case of error the handler first calls
|
||||||
|
nmdbus_libreswan_helper_complete_callback() which decreases the
|
||||||
|
refcount of "invocation", and then returns FALSE which tells the
|
||||||
|
skeleton code to return an error, also unreferencing the
|
||||||
|
invocation. This causes a crash.
|
||||||
|
|
||||||
|
Since the G_DBUS_METHOD_INVOCATION_HANDLED alias for TRUE is only
|
||||||
|
available since GLib 2.68 (while we target 2.36), just return TRUE.
|
||||||
|
|
||||||
|
Fixes: acb9eb9de50b ('service: process the configuration in the service, not the helper')
|
||||||
|
(cherry picked from commit 8ceb901719acac3778e1d76779d9c14289185157)
|
||||||
|
---
|
||||||
|
src/nm-libreswan-service.c | 3 ++-
|
||||||
|
1 file changed, 2 insertions(+), 1 deletion(-)
|
||||||
|
|
||||||
|
diff --git a/src/nm-libreswan-service.c b/src/nm-libreswan-service.c
|
||||||
|
index fc470a6..4850729 100644
|
||||||
|
--- a/src/nm-libreswan-service.c
|
||||||
|
+++ b/src/nm-libreswan-service.c
|
||||||
|
@@ -1379,7 +1379,8 @@ out:
|
||||||
|
}
|
||||||
|
|
||||||
|
nmdbus_libreswan_helper_complete_callback (object, invocation);
|
||||||
|
- return success;
|
||||||
|
+
|
||||||
|
+ return TRUE;
|
||||||
|
}
|
||||||
|
|
||||||
|
/****************************************************************/
|
||||||
|
--
|
||||||
|
2.43.0
|
||||||
|
|
@ -11,13 +11,14 @@
|
|||||||
Summary: NetworkManager VPN plug-in for IPsec VPN
|
Summary: NetworkManager VPN plug-in for IPsec VPN
|
||||||
Name: NetworkManager-libreswan
|
Name: NetworkManager-libreswan
|
||||||
Version: 1.2.10
|
Version: 1.2.10
|
||||||
Release: 4%{?dist}
|
Release: 5%{?dist}
|
||||||
License: GPLv2+
|
License: GPLv2+
|
||||||
URL: http://www.gnome.org/projects/NetworkManager/
|
URL: http://www.gnome.org/projects/NetworkManager/
|
||||||
Group: System Environment/Base
|
Group: System Environment/Base
|
||||||
Source0: https://download.gnome.org/sources/NetworkManager-libreswan/1.2/%{name}-%{version}.tar.xz
|
Source0: https://download.gnome.org/sources/NetworkManager-libreswan/1.2/%{name}-%{version}.tar.xz
|
||||||
Patch0: 0001-po-import-translations-from-Red-Hat-translators.patch
|
Patch0: 0001-po-import-translations-from-Red-Hat-translators.patch
|
||||||
Patch1: 0002-properties-set-advanced-dialog-modal.patch
|
Patch1: 0002-properties-set-advanced-dialog-modal.patch
|
||||||
|
Patch2: 0003-service-fix-wrong-refcounting-in-D-Bus-handler-for-C.patch
|
||||||
|
|
||||||
BuildRequires: gtk3-devel
|
BuildRequires: gtk3-devel
|
||||||
BuildRequires: libnl3-devel
|
BuildRequires: libnl3-devel
|
||||||
@ -109,6 +110,9 @@ update-desktop-database &> /dev/null || :
|
|||||||
%endif
|
%endif
|
||||||
|
|
||||||
%changelog
|
%changelog
|
||||||
|
* Mon Feb 5 2024 Wen Liang <wenliang@redhat.com> - 1.2.10-5
|
||||||
|
- Fix crash in libreswan_add_profile_wrong_password (RHEL-13123)
|
||||||
|
|
||||||
* Tue Jul 9 2019 Francesco Giudici <fgiudici@redhat.com> - 1.2.10-4
|
* Tue Jul 9 2019 Francesco Giudici <fgiudici@redhat.com> - 1.2.10-4
|
||||||
- Fix Gnome IPsec advanced options dialog (rh #1697329)
|
- Fix Gnome IPsec advanced options dialog (rh #1697329)
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user