dbus code provide an environment variable to DBUS_CONNECTION_TRACE to dump connection referencing like shown below. 13465: [dbus/dbus-internals.c(521):_dbus_trace_ref] DBusConnection 0x1490e00 3 -> 4 refs (ref_unlocked)[dbus/dbus-internals.c(521):_dbus_trace_ref] This output misses a newline at the end of each reference dump. The expected output format is shown below: 30122: [dbus/dbus-internals.c(521):_dbus_trace_ref] DBusConnection 0x1e1cf50 6 -> 5 refs (unref_unlocked) 30122: [dbus/dbus-internals.c(521):_dbus_trace_ref] DBusConnection 0x1e1cf50 5 -> 6 refs (ref_unlocked) 30122: [dbus/dbus-internals.c(521):_dbus_trace_ref] DBusConnection 0x1e1cf50 6 -> 5 refs (unref) The appended patch fixes this issue.
(In reply to Ralf Habacker from comment #0) > The appended patch fixes this issue. You forgot to attach the patch?
(In reply to Simon McVittie from comment #1) > (In reply to Ralf Habacker from comment #0) > > The appended patch fixes this issue. > > You forgot to attach the patch? Seems to be a bug in bugzilla :-( I added a patch on bug creation. Will readd later.
Created attachment 115055 [details] [review] Add newline to verbose output in _dbus_trace_ref().
Created attachment 115056 [details] [review] Fix 'dbus connection referencing issue in test-relay'.
Comment on attachment 115056 [details] [review] Fix 'dbus connection referencing issue in test-relay'. added to wrong bug :-(
Comment on attachment 115055 [details] [review] Add newline to verbose output in _dbus_trace_ref(). Review of attachment 115055 [details] [review]: ----------------------------------------------------------------- Yes please!
Comment on attachment 115055 [details] [review] Add newline to verbose output in _dbus_trace_ref(). committed to dbus-1.8
(In reply to Ralf Habacker from comment #7) > committed to dbus-1.8 This is such a trivial change that I'm probably not going to revert it, but in future please avoid committing stuff like this on stable-branches. Every change in a stable-branch is something that Linux distributions' QA people have to look at when deciding whether to approve a new version as "yes, this is a minimal fix"; and the more changes there are, the more likely they are to decide to do their own mini-fork of dbus instead, which hurts everyone in the long run. I would like change-averse distributions to be able to follow our stable branches.
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.