When checking if the new desired device flags are already pending, we
should compare them against the XOR of current flags and desired flags,
i.e. the flags that are going to change.
For example, let's assume the following situation:
- We have a device with `current_flags == DEVICE_FLAG_REMOTE_WAKEUP`
(i.e. 1)
- We want to disable the `wake_allowed` property
- `device_set_wake_allowed()` will call `adapter_set_device_flags()`
with `flags == 0`
- When in `adapter_set_device_flags()`, we'll have:
- current == 1
- pending == 0
- flags == 0
In this situation `flags == (flags & pending)` would not return what
we'd expect.
---
src/adapter.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/adapter.c b/src/adapter.c
index 5d4117a49..e55fb7f3b 100644
--- a/src/adapter.c
+++ b/src/adapter.c
@@ -5684,6 +5684,7 @@ void adapter_set_device_flags(struct btd_adapter *adapter,
mgmt_request_func_t func, void *user_data)
{
struct mgmt_cp_set_device_flags cp;
+ uint32_t current = btd_device_get_current_flags(device);
uint32_t supported = btd_device_get_supported_flags(device);
uint32_t pending = btd_device_get_pending_flags(device);
const bdaddr_t *bdaddr;
@@ -5694,7 +5695,7 @@ void adapter_set_device_flags(struct btd_adapter *adapter,
return;
/* Check if changing flags are pending */
- if (flags == (flags & pending))
+ if ((current ^ flags) == (flags & pending))
return;
/* Set Device Privacy Mode if it has not set the flag yet. */
--
2.48.1
