diff options
author | Havoc Pennington <hp@redhat.com> | 2006-06-14 04:16:30 +0000 |
---|---|---|
committer | Havoc Pennington <hp@redhat.com> | 2006-06-14 04:16:30 +0000 |
commit | 71308df778e5555383a559240e80f63ad1d1bee0 (patch) | |
tree | 8cf06502e6e2b688887809946d2da1a4a6595dbd | |
parent | 41fc87f6c504116db7e825bfbb1008368061e519 (diff) |
add dbus_bus_get refcount item to TODO
link to mail list post about pending call threading problems
-rw-r--r-- | doc/TODO | 12 |
1 files changed, 11 insertions, 1 deletions
@@ -21,7 +21,12 @@ Important for 1.0 - just before 1.0, try a HAVE_INT64=0 build and be sure it runs - - dbus-pending-call.c has some API and thread safety issues to review + - dbus-pending-call.c has some API and thread safety issues to review. + DBusPendingCall is used from multiple threads with no locks. + Either DBusConnection's lock has to protect all associated pending + call (means pending->connection can't ever be set to null) or + or DBusPendingCall needs its own lock + http://lists.freedesktop.org/archives/dbus/2006-June/004945.html - Add test harness for selinux allow/deny cf. this message http://lists.freedesktop.org/archives/dbus/2005-April/002506.html @@ -35,6 +40,11 @@ Important for 1.0 Kind of a major API change, but seems high-value. + - dbus_bus_get() should hold a strong reference associated with the "connected" + state (i.e. libdbus drops its reference when the connection disconnects, + and sets its internal connection variable to null). + See http://lists.freedesktop.org/archives/dbus/2006-May/004806.html + Important for 1.0 GLib Bindings === |