public inbox for git-commits@fedoraproject.org
help / color / mirror / Atom feed
From: Kalev Lember <klember@redhat.com>
To: git-commits@fedoraproject.org
Subject: [rpms/glib2] cve-2026-58016-f44: Fix network state monitoring to work with NetworkManager 1.32 (#1991075)
Date: Tue, 11 Aug 2026 10:37:35 GMT	[thread overview]
Message-ID: <178644465518.1.30047382685687034.rpms-glib2-a2b6df263a74@fedoraproject.org> (raw)

A new commit has been pushed.

Repo   : rpms/glib2
Branch : cve-2026-58016-f44
Commit : a2b6df263a7429ca91deba118914a607512f3596
Author : Kalev Lember <klember@redhat.com>
Date   : 2021-10-15T12:20:55+02:00
Stats  : +91/-0 in 2 file(s)
URL    : https://src.fedoraproject.org/rpms/glib2/c/a2b6df263a7429ca91deba118914a607512f3596?branch=cve-2026-58016-f44

Log:
Fix network state monitoring to work with NetworkManager 1.32 (#1991075)

---
diff --git a/0001-gnetworkmonitornm-Avoid-using-deprecated-PropertiesC.patch b/0001-gnetworkmonitornm-Avoid-using-deprecated-PropertiesC.patch
new file mode 100644
index 0000000..112e4a5
--- /dev/null
+++ b/0001-gnetworkmonitornm-Avoid-using-deprecated-PropertiesC.patch
@@ -0,0 +1,87 @@
+From f4e14edf66871c1b3dc213c286b28fd4a1742b78 Mon Sep 17 00:00:00 2001
+From: Kalev Lember <klember@redhat.com>
+Date: Fri, 15 Oct 2021 06:01:53 -0400
+Subject: [PATCH] gnetworkmonitornm: Avoid using deprecated PropertiesChanged
+ signal
+
+Instead, use the standard PropertiesChanged signal from
+org.freedesktop.DBus.Properties which exists since NetworkManager 1.2.0.
+
+This fixes the network state monitoring to correctly work with
+NetworkManager 1.32 that completely removed the deprecated
+org.freedesktop.NetworkManager PropertiesChanged signal.
+---
+ gio/gnetworkmonitornm.c | 49 +++++------------------------------------
+ 1 file changed, 6 insertions(+), 43 deletions(-)
+
+diff --git a/gio/gnetworkmonitornm.c b/gio/gnetworkmonitornm.c
+index 5a36a0ba1..a8040fb36 100644
+--- a/gio/gnetworkmonitornm.c
++++ b/gio/gnetworkmonitornm.c
+@@ -253,48 +253,11 @@ sync_properties (GNetworkMonitorNM *nm,
+ }
+ 
+ static void
+-update_cached_property (GDBusProxy   *proxy,
+-                        const char   *property_name,
+-                        GVariantDict *dict)
++proxy_properties_changed_cb (GDBusProxy        *proxy,
++                             GVariant          *changed_properties,
++                             GStrv              invalidated_properties,
++                             GNetworkMonitorNM *nm)
+ {
+-  GVariant *v;
+-
+-  v = g_variant_dict_lookup_value (dict, property_name, NULL);
+-  if (!v)
+-    return;
+-  g_dbus_proxy_set_cached_property (proxy, property_name, v);
+-  g_variant_unref (v);
+-}
+-
+-static void
+-proxy_signal_cb (GDBusProxy        *proxy,
+-                 const gchar       *sender_name,
+-                 const gchar       *signal_name,
+-                 GVariant          *parameters,
+-                 GNetworkMonitorNM *nm)
+-{
+-  GVariant *asv;
+-  GVariantDict *dict;
+-
+-  if (g_strcmp0 (signal_name, "PropertiesChanged") != 0)
+-    return;
+-
+-  g_variant_get (parameters, "(@a{sv})", &asv);
+-  if (!asv)
+-    return;
+-
+-  dict = g_variant_dict_new (asv);
+-  g_variant_unref (asv);
+-  if (!dict)
+-    {
+-      g_warning ("Failed to handle PropertiesChanged signal from NetworkManager");
+-      return;
+-    }
+-
+-  update_cached_property (nm->priv->proxy, "Connectivity", dict);
+-
+-  g_variant_dict_unref (dict);
+-
+   sync_properties (nm, TRUE);
+ }
+ 
+@@ -361,8 +324,8 @@ g_network_monitor_nm_initable_init (GInitable     *initable,
+       return FALSE;
+     }
+ 
+-  nm->priv->signal_id = g_signal_connect (G_OBJECT (proxy), "g-signal",
+-                                          G_CALLBACK (proxy_signal_cb), nm);
++  nm->priv->signal_id = g_signal_connect (G_OBJECT (proxy), "g-properties-changed",
++                                          G_CALLBACK (proxy_properties_changed_cb), nm);
+   nm->priv->proxy = proxy;
+   sync_properties (nm, FALSE);
+ 
+-- 
+2.31.1
+

diff --git a/glib2.spec b/glib2.spec
index 5eff3ec..7478af4 100644
--- a/glib2.spec
+++ b/glib2.spec
@@ -17,6 +17,10 @@ Patch0: gnutls-hmac.patch
 # Proposed upstream at https://gitlab.gnome.org/GNOME/glib/-/merge_requests/1596
 Patch1: gdesktopappinfo.patch
 
+# https://bugzilla.redhat.com/show_bug.cgi?id=1991075
+# https://gitlab.gnome.org/GNOME/glib/-/merge_requests/2296
+Patch2: 0001-gnetworkmonitornm-Avoid-using-deprecated-PropertiesC.patch
+
 BuildRequires: chrpath
 BuildRequires: gcc
 BuildRequires: gcc-c++

                 reply	other threads:[~2026-08-11 10:37 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=178644465518.1.30047382685687034.rpms-glib2-a2b6df263a74@fedoraproject.org \
    --to=klember@redhat.com \
    --cc=git-commits@fedoraproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox