|
 |
fc9aca |
From 4bf6a4dd5b73fcea36fcfa4a97091665b233bc15 Mon Sep 17 00:00:00 2001
|
|
|
fc9aca |
From: Thomas Haller <thaller@redhat.com>
|
|
|
fc9aca |
Date: Thu, 11 May 2017 12:09:45 +0200
|
|
|
fc9aca |
Subject: [PATCH 1/2] proxy: fix refcount handing for DestroyProxyConfiguration
|
|
|
fc9aca |
operation
|
|
|
fc9aca |
|
|
|
fc9aca |
Fixes: e895beb0da38fc87ce93fe7403a6b50e92f0dd82
|
|
|
fc9aca |
(cherry picked from commit df137fdf9a7077c20d8923f068568c22fb479e5a)
|
|
|
fc9aca |
(cherry picked from commit 10373de9072f0318ea1150c85fd7b7ebfa5a9655)
|
|
|
fc9aca |
---
|
|
|
fc9aca |
src/nm-pacrunner-manager.c | 1 +
|
|
|
fc9aca |
1 file changed, 1 insertion(+)
|
|
|
fc9aca |
|
|
|
fc9aca |
diff --git a/src/nm-pacrunner-manager.c b/src/nm-pacrunner-manager.c
|
|
|
fc9aca |
index cfc028c..fd3d1f8 100644
|
|
|
fc9aca |
--- a/src/nm-pacrunner-manager.c
|
|
|
fc9aca |
+++ b/src/nm-pacrunner-manager.c
|
|
|
fc9aca |
@@ -250,6 +250,7 @@ pacrunner_send_done (GDBusProxy *proxy, GAsyncResult *res, gpointer user_data)
|
|
|
fc9aca |
_LOG2D (config, "sent");
|
|
|
fc9aca |
|
|
|
fc9aca |
if (config->removed) {
|
|
|
fc9aca |
+ config_ref (config);
|
|
|
fc9aca |
g_dbus_proxy_call (priv->pacrunner,
|
|
|
fc9aca |
"DestroyProxyConfiguration",
|
|
|
fc9aca |
g_variant_new ("(o)", config->path),
|
|
|
fc9aca |
--
|
|
|
fc9aca |
2.9.3
|
|
|
fc9aca |
|
|
|
fc9aca |
|
|
|
fc9aca |
From 617aa8fd2fccbd8a8cb76fbf9bce3e74eac78f8c Mon Sep 17 00:00:00 2001
|
|
|
fc9aca |
From: Thomas Haller <thaller@redhat.com>
|
|
|
fc9aca |
Date: Thu, 11 May 2017 12:32:22 +0200
|
|
|
fc9aca |
Subject: [PATCH 2/2] proxy: fix passing cancellable to async D-Bus operations
|
|
|
fc9aca |
|
|
|
fc9aca |
We must not cancel pacrunner_cancellable when the D-Bus proxy is
|
|
|
fc9aca |
created. Instead, keep it around and use it later for the asynchronous
|
|
|
fc9aca |
D-Bus operations.
|
|
|
fc9aca |
|
|
|
fc9aca |
This doesn't really matter at the moment, because the pacrunner manager
|
|
|
fc9aca |
is only destroyed when NetworkManager is about to terminated. That is
|
|
|
fc9aca |
the only time when we actually cancel the asynchronous request. Also,
|
|
|
fc9aca |
at that time we no longer iterate the mainloop, so the pending requests
|
|
|
fc9aca |
are never completed anyway.
|
|
|
fc9aca |
|
|
|
fc9aca |
(cherry picked from commit a08540d967812457af192ebd34497412da5d143e)
|
|
|
fc9aca |
(cherry picked from commit 6cfd9279625366c24808d86e1c3b04a18a036eb6)
|
|
|
fc9aca |
---
|
|
|
fc9aca |
src/nm-pacrunner-manager.c | 1 -
|
|
|
fc9aca |
1 file changed, 1 deletion(-)
|
|
|
fc9aca |
|
|
|
fc9aca |
diff --git a/src/nm-pacrunner-manager.c b/src/nm-pacrunner-manager.c
|
|
|
fc9aca |
index fd3d1f8..87e0a36 100644
|
|
|
fc9aca |
--- a/src/nm-pacrunner-manager.c
|
|
|
fc9aca |
+++ b/src/nm-pacrunner-manager.c
|
|
|
fc9aca |
@@ -330,7 +330,6 @@ pacrunner_proxy_cb (GObject *source, GAsyncResult *res, gpointer user_data)
|
|
|
fc9aca |
priv = NM_PACRUNNER_MANAGER_GET_PRIVATE (self);
|
|
|
fc9aca |
|
|
|
fc9aca |
priv->pacrunner = proxy;
|
|
|
fc9aca |
- nm_clear_g_cancellable (&priv->pacrunner_cancellable);
|
|
|
fc9aca |
|
|
|
fc9aca |
g_signal_connect (priv->pacrunner, "notify::g-name-owner",
|
|
|
fc9aca |
G_CALLBACK (name_owner_changed_cb), self);
|
|
|
fc9aca |
--
|
|
|
fc9aca |
2.9.3
|
|
|
fc9aca |
|