[at-spi2-core: 3/4] at-spi-bus-launcher: defer starting atspi bus after bus name acquired

[Mike Gorse <mgorse src gnome org>]

commit 018443b7a8ad0b6f068c2255adcb49037a388d15
Author: Samuel Thibault <samuel thibault ens-lyon org>
Date:   Fri Sep 6 03:40:53 2019 +0200

    at-spi-bus-launcher: defer starting atspi bus after bus name acquired
    
    In case bus name acquisition fails, we should not have started a bus
    after all, but worse, we should not have written its address in the
    AT_SPI_BUS X root property. We should thus do them only after having
    acquired the bus name.

 bus/at-spi-bus-launcher.c | 22 ++++++++++++----------
 1 file changed, 12 insertions(+), 10 deletions(-)
---
diff --git a/bus/at-spi-bus-launcher.c b/bus/at-spi-bus-launcher.c
index af85f04..b4f49b8 100644
--- a/bus/at-spi-bus-launcher.c
+++ b/bus/at-spi-bus-launcher.c
@@ -683,16 +683,6 @@ on_bus_acquired (GDBusConnection *connection,
     }
   app->session_bus = connection;
 
-  if (app->launch_immediately)
-    {
-      ensure_a11y_bus (app);
-      if (app->state == A11Y_BUS_STATE_ERROR)
-        {
-          g_main_loop_quit (app->loop);
-          return;
-        }
-    }
-
   error = NULL;
   registration_id = g_dbus_connection_register_object (connection,
                                                        "/org/a11y/bus",
@@ -734,6 +724,18 @@ on_name_acquired (GDBusConnection *connection,
                   const gchar     *name,
                   gpointer         user_data)
 {
+  A11yBusLauncher *app = user_data;
+
+  if (app->launch_immediately)
+    {
+      ensure_a11y_bus (app);
+      if (app->state == A11Y_BUS_STATE_ERROR)
+        {
+          g_main_loop_quit (app->loop);
+          return;
+        }
+    }
+
   g_bus_watch_name (G_BUS_TYPE_SESSION,
                     "org.gnome.SessionManager",
                     G_BUS_NAME_WATCHER_FLAGS_NONE,