diff --git a/NetworkManager.spec b/NetworkManager.spec index 90b28207ba211fc6d6f1462f88b71bd29b97b38c..0da30e0168248d91a43bdae24dc765610c7cf2c2 100644 --- a/NetworkManager.spec +++ b/NetworkManager.spec @@ -50,7 +50,7 @@ Name: NetworkManager Version: 1.26.2 Epoch: 1 -Release: 2 +Release: 3 Summary: Network Link Manager and User Applications License: GPLv2+ URL: https://www.gnome.org/projects/NetworkManager/ @@ -66,6 +66,7 @@ Patch5: backport-wwan-fix-leaking-bearer-in-connect-ready.patch Patch6: backport-iwd-Fix-a-use-after-free.patch Patch7: backport-core-fix-crash-in-nm_wildcard_match_check.patch Patch8: backport-core-ovs-fix-leak-of-NMOvsdbPrivate-db_uuid.patch +Patch9: backport-libnm-fix-leak-and-return-failures-from-nm-client-load-connections.patch BuildRequires: gcc libtool pkgconfig automake autoconf intltool gettext-devel ppp-devel gnutls-devel BuildRequires: dbus-devel dbus-glib-devel glib2-devel gobject-introspection-devel jansson-devel @@ -476,6 +477,12 @@ fi %{_datadir}/gtk-doc/html/NetworkManager/* %changelog +* Mon Feb 7 2022 chxssg - 1.26.2-3 +- Type:bugfix +- ID:NA +- SUG:NA +- DESC:libnm: fix leak and return failures from nm_client_load_connections() + * Sat Dec 18 2021 gaoxingwang - 1.26.2-2 - Type:bugfix - ID:NA diff --git a/backport-libnm-fix-leak-and-return-failures-from-nm-client-load-connections.patch b/backport-libnm-fix-leak-and-return-failures-from-nm-client-load-connections.patch new file mode 100644 index 0000000000000000000000000000000000000000..2b2fe5ed13641fd2113c9041d2db81cad81cf349 --- /dev/null +++ b/backport-libnm-fix-leak-and-return-failures-from-nm-client-load-connections.patch @@ -0,0 +1,51 @@ +From 2a36f8c2f157299cf8ffb81703b044c49ad9b1fd Mon Sep 17 00:00:00 2001 +From: gaoxingwang +Date: Mon, 13 Sep 2021 11:00:13 +0800 +Subject: [PATCH] libnm: fix leak and return "failures" from + nm_client_load_connections() + +Due to this, `nmcli connection load` would also not print a warning +about failure to load obviously bogus files: + + $ nmcli connection load /bogus + +Note that load is also used to unload files, so if the file name is a +possibly valid name for a non-existing file, there is no failure. For +example, we get no warning for + + $ nmcli connection load /etc/NetworkManager/system-connections/bogus + +Even if currently no such file is loaded, then the operation would still +silently succeed, instead of succeeding the first time only. That is because +load should be idempotent. + +[thaller@redhat.com: rewrote commit message] + +Fixes: 4af6219226e0 ('libnm: implement nm_client_load_connections_async() by using GDBusConnection directly') + +https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/issues/794 + +https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/merge_requests/979 + +Conflict:apply to libnm/nm-client.c +Reference:https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/commit/2a36f8c2f157299cf8ffb81703b044c49ad9b1fd +--- +--- + libnm/nm-client.c | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +diff --git a/libnm/nm-client.c b/libnm/nm-client.c +index d7f6115..27c5f39 100644 +--- a/libnm/nm-client.c ++++ b/libnm/nm-client.c +@@ -5850,7 +5850,7 @@ nm_client_load_connections (NMClient *client, + g_variant_get (ret, + "(b^as)", + NULL, +- &failures); ++ failures); + + return TRUE; + } +-- +2.27.0