Exit without a SIGABRT when unable to open X display
authorMike Gorse <mgorse@suse.com>
Mon, 10 Sep 2012 17:00:19 +0000 (12:00 -0500)
committerMike Gorse <mgorse@suse.com>
Mon, 10 Sep 2012 17:00:19 +0000 (12:00 -0500)
At-spi2-registryd was calling g_error when unable to open the default
display, which led to a SIGABRT and a core dump. This behavior is
inconsistent with what is generally done by GNOME apps, so let's call
g_warning() and exit(1) instead.

https://bugzilla.gnome.org/show_bug.cgi?id=660407

registryd/display.c

index 730b61c..2ba781c 100644 (file)
@@ -43,8 +43,8 @@ Display *spi_set_display (const char *display_name)
         default_display = XOpenDisplay (display_name);  
         if (!default_display)
         {
-                g_error ("AT-SPI: Cannot open default display");
-                return NULL;
+                g_warning ("AT-SPI: Cannot open default display");
+                exit (1);
         }
  return default_display;
 }