Fix parsing of "Flags" AD type
If an advertising report did not contain a "Flags" AD Type (e.g. a Scan
response), the previous flags value was mistakenly set to 0x00. This fix
makes sure dev->flags is only updated for valid values.
diff --git a/src/adapter.c b/src/adapter.c
index fd4b685..f55cd27 100644
--- a/src/adapter.c
+++ b/src/adapter.c
@@ -3032,7 +3032,7 @@
void adapter_update_device_from_info(struct btd_adapter *adapter,
bdaddr_t bdaddr, int8_t rssi,
uint8_t evt_type, const char *name,
- GSList *services, uint8_t flags)
+ GSList *services, int flags)
{
struct remote_dev_info *dev;
gboolean new_dev;
@@ -3053,7 +3053,8 @@
g_slist_foreach(services, remove_same_uuid, dev);
g_slist_foreach(services, dev_prepend_uuid, dev);
- dev->flags = flags;
+ if (flags >= 0)
+ dev->flags = flags;
if (name) {
g_free(dev->name);
diff --git a/src/adapter.h b/src/adapter.h
index 1edc441..39dd905 100644
--- a/src/adapter.h
+++ b/src/adapter.h
@@ -135,7 +135,7 @@
void adapter_update_device_from_info(struct btd_adapter *adapter,
bdaddr_t bdaddr, int8_t rssi,
uint8_t evt_type, const char *name,
- GSList *services, uint8_t flags);
+ GSList *services, int flags);
void adapter_update_found_devices(struct btd_adapter *adapter, bdaddr_t *bdaddr,
int8_t rssi, uint32_t class, const char *name,
const char *alias, gboolean legacy,
diff --git a/src/event.c b/src/event.c
index 0659fd1..6f345d4 100644
--- a/src/event.c
+++ b/src/event.c
@@ -60,7 +60,7 @@
struct eir_data {
GSList *services;
- uint8_t flags;
+ int flags;
char *name;
gboolean name_complete;
};
@@ -289,6 +289,8 @@
char *uuid_str;
unsigned int i;
+ eir->flags = -1;
+
/* No EIR data to parse */
if (eir_data == NULL || eir_length == 0)
return 0;