diff options
| author | Johan Gyllenspetz <johangy@axis.com> | 2009-03-17 17:26:03 -0400 |
|---|---|---|
| committer | Colin Walters <walters@verbum.org> | 2009-05-06 12:51:17 -0400 |
| commit | 5b4ee5fb40269afaa106b55dd4755125c2f9107a (patch) | |
| tree | fee42ebaeccca5ce302d62e24106538101792870 | |
| parent | 15f518301605ed748fbcecdf5e38d0a5ef982c3b (diff) | |
| download | dbus-5b4ee5fb40269afaa106b55dd4755125c2f9107a.tar.gz | |
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 <walters@verbum.org>
| -rw-r--r-- | dbus/dbus-connection.c | 4 |
1 files changed, 2 insertions, 2 deletions
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); |