Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • pulseaudio pulseaudio
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 863
    • Issues 863
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 49
    • Merge requests 49
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • PulseAudio
  • pulseaudiopulseaudio
  • Merge requests
  • !472

bluetooth/bluez5-util: Do not reply to ClearConfiguration

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Marijn Suijten requested to merge MarijnS95/pulseaudio:silence-bluez5-no-method-return into master Jan 18, 2021
  • Overview 2
  • Commits 1
  • Pipelines 2
  • Changes 1

Similar to the situation/comment in endpoint_release BlueZ does not request any reply to ClearConfiguration() either; sending one results in the same "0 matched rules" warning from dbus-daemon:

dbus-daemon[1309]: [system] Rejected send message, 0 matched rules; type="method_return", sender=":1.71" (uid=1000 pid=87548 comm="../build/src/daemon/pulseaudio -vvvv -n -F ../buil") interface="(unset)" member="(unset)" error name="(unset)" requested_reply="0" destination=":1.3" (uid=0 pid=1308 comm="/usr/lib/bluetooth/bluetoothd -d ")

Solve this by only creating a return message when an (othwise empty) reply is solicited for, just like in endpoint_release.

Unfortunately we also have to make sure to not send any error back if no reply is requested, but fortunately an argument parsing error here is extremely unlikely.


This came up during testing of !440 (merged) but is not touched directly at all in that MR; hence a separate MR to not crowd review comments and diff even further.

Not sure if we should keep the error reply in though, seems very unlikely for BlueZ to suddenly accept a reply here and perform alternative logic (but if it happens, we're ready for it at least).

CC @SanchayanMaity

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: silence-bluez5-no-method-return