Warn if a device listener returns a non-bool, rather than crash
authorMike Gorse <mgorse@novell.com>
Thu, 29 Dec 2011 14:23:10 +0000 (08:23 -0600)
committerMike Gorse <mgorse@novell.com>
Thu, 29 Dec 2011 14:47:05 +0000 (08:47 -0600)
atspi/atspi-device-listener.c

index 1e68eb1..58a6d6a 100644 (file)
@@ -389,6 +389,11 @@ _atspi_dbus_handle_DeviceEvent (DBusConnection *bus, DBusMessage *message, void
   if (klass->device_event)
   {
     retval = (*klass->device_event) (listener, &event);
+    if (retval != 0 && retval != 1)
+    {
+      g_warning ("at-spi: device event handler returned %d; should be 0 or 1", retval);
+      retval = 0;
+    }
   }
 done:
   reply = dbus_message_new_method_return (message);