Hi Andrew, On 03/06/2025 17:13, Andrew Sayers wrote:
DBus requests are received by the public DBus connection, because it registers the relevant bus name. My previous patches told PBAP to listen on a private connection, but DBus doesn't let connections share bus names, so it needs to listen on the public connection instead. This adds a little more complexity to the logind callbacks, to avoid unregistering profiles during shutdown. It also removes an API I previously added, which is no longer used and would only encourage the next person to make the same mistake. Reported-by: Frédéric Danis <frederic.danis@xxxxxxxxxxxxx> Closes: https://lore.kernel.org/linux-bluetooth/333ad76e-0aba-4f93-b141-8e69fb47535f@xxxxxxxxxxxxx/ Suggested-by: Pauli Virtanen <pav@xxxxxx> Signed-off-by: Andrew Sayers <kernel.org@xxxxxxxxxxxxxxx> --- Andrew Sayers (3): obexd: Pass at_(un)register value to logind callbacks pbap: use the public DBus connection Revert "obexd: Support creating private system/session bus connections" obexd/client/pbap.c | 28 ++++++++++++++++++++++++---- obexd/plugins/bluetooth.c | 6 ++++-- obexd/src/logind.c | 14 +++++++------- obexd/src/logind.h | 8 ++++---- obexd/src/main.c | 8 -------- obexd/src/obexd.h | 2 -- 6 files changed, 39 insertions(+), 27 deletions(-)
Except 2 simple build errors, this allows me to connect the pbap profile. Thanks -- Frédéric Danis Senior Software Engineer Collabora Ltd. Platinum Building, St John's Innovation Park, Cambridge CB4 0DS, United Kingdom Registered in England & Wales, no. 5513718
