Skip to content

Commit

Permalink
bluez5: Fix invalid arguments to PairDevice
Browse files Browse the repository at this point in the history
The third (device class) argument to PairDevice was removed in 0.30.1,
but this call to it was still passing a third parameter, resulting in
an error from dbus-python whenever Pair() was called. This caused a
unit test regression in gnome-bluetooth.

Fixes: 63264e1 "bluez5: Clean up static default properties, re-drop PairDevice class_ parameter"
Fixes #193
Bug-Debian: https://bugs.debian.org/1057564

Signed-off-by: Simon McVittie <smcv@debian.org>
  • Loading branch information
smcv authored Dec 31, 2023
1 parent e991224 commit d70b68c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion dbusmock/templates/bluez5.py
Original file line number Diff line number Diff line change
Expand Up @@ -333,7 +333,7 @@ def Pair(device):
raise dbus.exceptions.DBusException("Device already paired", name="org.bluez.Error.AlreadyExists")
device_address = device.props[DEVICE_IFACE]["Address"]
adapter_device_name = Path(device.props[DEVICE_IFACE]["Adapter"]).name
device.PairDevice(adapter_device_name, device_address, MOCK_PHONE_CLASS)
device.PairDevice(adapter_device_name, device_address)


@dbus.service.method(DEVICE_IFACE, in_signature="", out_signature="")
Expand Down

0 comments on commit d70b68c

Please sign in to comment.