summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorThomas Haller <thaller@redhat.com>2014-08-24 15:59:39 +0200
committerThomas Haller <thaller@redhat.com>2014-08-27 17:29:06 +0200
commit1731d6db5176f17f88fdcc48f99916101b8a12ae (patch)
treec802dee7df2b077872c44ad282ab3c0af4047ad5
parentc83d850149784b8ecab59008d85df4cd0e7ada14 (diff)
downloadnetwork-manager-applet-1731d6db5176f17f88fdcc48f99916101b8a12ae.tar.gz
applet: ensure to create the secret agent instance only once
Since we don't disconnect shell_version_changed_cb(), make sure that we don't call register_agent() when an agent is already registered. https://bugzilla.gnome.org/show_bug.cgi?id=727923 Signed-off-by: Thomas Haller <thaller@redhat.com>
-rw-r--r--src/applet.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/src/applet.c b/src/applet.c
index 9cab4936..f4817032 100644
--- a/src/applet.c
+++ b/src/applet.c
@@ -3533,6 +3533,8 @@ applet_embedded_cb (GObject *object, GParamSpec *pspec, gpointer user_data)
static void
register_agent (NMApplet *applet)
{
+ g_return_if_fail (!applet->agent);
+
applet->agent = applet_agent_new ();
g_assert (applet->agent);
g_signal_connect (applet->agent, APPLET_AGENT_GET_SECRETS,
@@ -3568,7 +3570,7 @@ shell_version_changed_cb (NMShellWatcher *watcher, GParamSpec *pspec, gpointer u
G_CALLBACK (shell_version_changed_cb),
applet);
g_clear_object (&applet->shell_watcher);
- } else {
+ } else if (!applet->agent) {
/* No shell */
g_debug ("gnome-shell is not running, registering secret agent");
register_agent (applet);