From 387ccafb4df116bdcc7f4a07cd0098a1f445b958 Mon Sep 17 00:00:00 2001 From: Johan Gyllenspetz Date: Tue, 17 Mar 2009 17:26:03 -0400 Subject: Bug 20494 - Fix signed confusion for dbus_message_get_reply_serial return We were incorrectly converting the serial to a signed integer and comparing it to -1. Signed-off-by: Colin Walters --- dbus/dbus-connection.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'dbus/dbus-connection.c') diff --git a/dbus/dbus-connection.c b/dbus/dbus-connection.c index a960a991..ae07adf0 100644 --- a/dbus/dbus-connection.c +++ b/dbus/dbus-connection.c @@ -448,7 +448,7 @@ _dbus_connection_queue_received_message_link (DBusConnection *connection, DBusList *link) { DBusPendingCall *pending; - dbus_int32_t reply_serial; + dbus_uint32_t reply_serial; DBusMessage *message; _dbus_assert (_dbus_transport_get_is_authenticated (connection->transport)); @@ -459,7 +459,7 @@ _dbus_connection_queue_received_message_link (DBusConnection *connection, /* If this is a reply we're waiting on, remove timeout for it */ reply_serial = dbus_message_get_reply_serial (message); - if (reply_serial != -1) + if (reply_serial != 0) { pending = _dbus_hash_table_lookup_int (connection->pending_replies, reply_serial); -- cgit