OSDN Git Service

Clean up check_sdp_bl
authorHansong Zhang <hsz@google.com>
Fri, 30 Oct 2020 03:58:53 +0000 (20:58 -0700)
committerHansong Zhang <hsz@google.com>
Thu, 5 Nov 2020 02:03:01 +0000 (18:03 -0800)
Some variables are unused or overriden

Bug: 159815595
Tag: #refactor
Test: compile & verify basic functions working
Change-Id: I3d9f24273d03540c9218e7ce61e6633d020eaba3

btif/src/btif_dm.cc

index eeae3f2..acc444f 100644 (file)
@@ -404,17 +404,11 @@ bool check_cod_hid(const RawAddress* remote_bdaddr) {
  *
  ******************************************************************************/
 bool check_sdp_bl(const RawAddress* remote_bdaddr) {
-  uint16_t manufacturer = 0;
-  uint8_t lmp_ver = 0;
-  uint16_t lmp_subver = 0;
   bt_property_t prop_name;
   bt_remote_version_t info;
 
   if (remote_bdaddr == NULL) return false;
 
-  /* fetch additional info about remote device used in iop query */
-  BTM_ReadRemoteVersion(*remote_bdaddr, &lmp_ver, &manufacturer, &lmp_subver);
-
   /* if not available yet, try fetching from config database */
   BTIF_STORAGE_FILL_PROPERTY(&prop_name, BT_PROPERTY_REMOTE_VERSION_INFO,
                              sizeof(bt_remote_version_t), &info);
@@ -423,7 +417,7 @@ bool check_sdp_bl(const RawAddress* remote_bdaddr) {
       BT_STATUS_SUCCESS) {
     return false;
   }
-  manufacturer = info.manufacturer;
+  uint16_t manufacturer = info.manufacturer;
 
   for (unsigned int i = 0; i < ARRAY_SIZE(sdp_blacklist); i++) {
     if (manufacturer == sdp_blacklist[i].manufact_id) return true;