dbus/bus dispatch.c,1.71,1.72

[Mark McLoughlin]

Update of /cvs/dbus/dbus/bus
In directory gabe:/tmp/cvs-serv30632/bus

Modified Files:
	dispatch.c 
Log Message:
2005-09-11  Mark McLoughlin  <mark at skynet.ie>

	* test/data/auth/fallback.auth-script: we don't
	retry the EXTERNAL method when we know its going
	to fail anymore.

2005-09-11  Mark McLoughlin  <mark at skynet.ie>

	* dbus/dbus-connection-internal.h: rename
	(add|remove|toggle)_(watch|timeout) to unlocked()
	
	* dbus/dbus-connection.c: ditto.
	
	* dbus/dbus-timeout.c, dbus/dbus-transport-unix.c:
	Update some callers for the renaming.

2005-09-10  Mark McLoughlin  <mark at skynet.ie>

	* dbus/dbus-auth.c: (record_mechanisms): don't
	retry the first auth mechanism because we know
	we're just going to get rejected again.
	
	* dbus/dbus-keyring.c: (_dbus_keyring_reload):
	Fix thinko ... and what a nasty little bugger to
	track down you were ...

	* dbus/dbus-connection.c:
	(_dbus_connection_add_watch),
	(_dbus_connection_remove_watch): add note about
	these needing the connection to be locked.
	(_dbus_connection_get_dispatch_status_unlocked):
	set status to DATA_REMAINS when we queue the
	disconnected message.
	
	* bus/dispatch.c:
	(bus_dispatch): fix warning.
	(check_existent_service_no_auto_start):
	Expect ChildSignaled error too.
	(check_existent_hello_from_self): fix another
	couple of warnings.
	


Index: dispatch.c
===================================================================
RCS file: /cvs/dbus/dbus/bus/dispatch.c,v
retrieving revision 1.71
retrieving revision 1.72
diff -u -d -r1.71 -r1.72
--- dispatch.c	15 Jul 2005 15:21:43 -0000	1.71
+++ dispatch.c	11 Sep 2005 10:02:47 -0000	1.72
@@ -271,8 +271,7 @@
                                                 message, service_name, &error))
             {
               _DBUS_ASSERT_ERROR_IS_SET (&error);
-              _dbus_verbose ("bus_activation_activate_service() failed\n");
-               ("Failed: %s\n", error.name);
+              _dbus_verbose ("bus_activation_activate_service() failed: %s\n", error.name);
               goto out;
             }
           
@@ -2475,6 +2474,8 @@
       else if (dbus_message_is_error (message,
                                       DBUS_ERROR_SPAWN_CHILD_EXITED) ||
                dbus_message_is_error (message,
+                                      DBUS_ERROR_SPAWN_CHILD_SIGNALED) ||
+               dbus_message_is_error (message,
                                       DBUS_ERROR_SPAWN_EXEC_FAILED))
         {
           ; /* good, this is expected also */
@@ -2821,8 +2822,6 @@
 {
   DBusMessage *message;
   dbus_uint32_t serial;
-  dbus_bool_t retval;
-  const char *base_service;
   const char *text;
 
   message = dbus_message_new_method_call (EXISTENT_SERVICE_NAME,
@@ -2862,25 +2861,20 @@
   if (message == NULL)
     {
       _dbus_warn ("Failed to pop message! Should have been reply from RunHelloFromSelf message\n");
-      goto out;
+      return FALSE;
     }
 
   if (dbus_message_get_reply_serial (message) != serial)
     {
       _dbus_warn ("Wrong reply serial\n");
-      goto out;
+      dbus_message_unref (message);
+      return FALSE;
     }
 
   dbus_message_unref (message);
   message = NULL;
       
-  retval = TRUE;
-
- out:
-  if (message)
-    dbus_message_unref (message);
-
-  return retval;
+  return TRUE;
 }