This fix two issues: - not storing attributes after pairing if device is using RPA - storing attributes for temporary device if it is using static or public address --- src/device.c | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/src/device.c b/src/device.c index b43e21658..ce515be9b 100644 --- a/src/device.c +++ b/src/device.c @@ -2340,11 +2340,12 @@ static void device_svc_resolved(struct btd_device *dev, uint8_t browse_type, dev->pending_paired = false; } - if (!dev->temporary) + if (!dev->temporary) { store_device_info(dev); - if (bdaddr_type != BDADDR_BREDR && err == 0) - store_services(dev); + if (bdaddr_type != BDADDR_BREDR && err == 0) + store_services(dev); + } if (req) browse_request_complete(req, browse_type, bdaddr_type, err); @@ -5289,6 +5290,12 @@ void btd_device_set_temporary(struct btd_device *device, bool temporary) adapter_whitelist_add(device->adapter, device); store_device_info(device); + + /* attributes were not stored when resolved if device was temporary */ + if (device->bdaddr_type != BDADDR_BREDR && + device->le_state.svc_resolved && + g_slist_length(device->primaries) != 0) + store_services(device); } void btd_device_set_trusted(struct btd_device *device, gboolean trusted) -- 2.14.3 -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html