2003-06-20 Anders Carlsson <andersca@codefactory.se>

	* dbus/dbus-transport-unix.c (unix_handle_watch): Check
	for hangup and error after checking read so we won't discard
	pending data if both hangup and read are set.

diff --git a/ChangeLog b/ChangeLog
index eb7cd6d2bae53f4da7f87bd7dd9023d2135815e2..c9343bf9baaf31c0caf246e2e0dd68992f5acf04 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2003-06-20  Anders Carlsson  <andersca@codefactory.se>
+
+       * dbus/dbus-transport-unix.c (unix_handle_watch): Check 
+       for hangup and error after checking read so we won't discard
+       pending data if both hangup and read are set.
+
 2003-06-19  Philip Blundell  <philb@gnu.org>
 
        * tools/dbus-print-message.c (print_message): Handle BOOLEAN.

diff --git a/dbus/dbus-transport-unix.c b/dbus/dbus-transport-unix.c
index e5e446c26c48b582ccfb456c696fe97c6607c112..40c7a5dffd46a48fab6e71648a0c5f3e63bb2feb 100644 (file)
--- a/dbus/dbus-transport-unix.c
+++ b/dbus/dbus-transport-unix.c
@@ -727,12 +727,6 @@ unix_handle_watch (DBusTransport *transport,
   _dbus_assert (watch == unix_transport->read_watch ||
                 watch == unix_transport->write_watch);
   
-  if (flags & (DBUS_WATCH_HANGUP | DBUS_WATCH_ERROR))
-    {
-      _dbus_transport_disconnect (transport);
-      return TRUE;
-    }
-  
   if (watch == unix_transport->read_watch &&
       (flags & DBUS_WATCH_READABLE))
     {
@@ -779,6 +773,12 @@ unix_handle_watch (DBusTransport *transport,
     }
 #endif /* DBUS_ENABLE_VERBOSE_MODE */
 
+  if (flags & (DBUS_WATCH_HANGUP | DBUS_WATCH_ERROR))
+    {
+      _dbus_transport_disconnect (transport);
+      return TRUE;
+    }
+  
   return TRUE;
 }