commit ba6972f30e6d05a7800be56e056a2cbcc7369479
author Hansong Zhang <hsz@google.com> Thu Oct 29 19:45:32 2020 -0700
committer Hansong Zhang <hsz@google.com> Fri Oct 30 10:22:53 2020 -0700
tree 9e8ae5d116df4faa864bcfa9ee7f315015e3fcbc
parent 95467cf42b35e7b82af488ee5c3bacbc7879c1f5

BTM_SecAddDevice: features is always nullptr

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

stack/btm/btm_dev.cc

diff --git a/stack/btm/btm_dev.cc b/stack/btm/btm_dev.cc
index 116e5d7..98b5c18 100644
--- a/stack/btm/btm_dev.cc
+++ b/stack/btm/btm_dev.cc
@@ -100,22 +100,7 @@
   }
 
   p_dev_rec->num_read_pages = 0;
-  if (features) {
-    bool found = false;
-    memcpy(p_dev_rec->feature_pages, features,
-           sizeof(p_dev_rec->feature_pages));
-    for (int i = HCI_EXT_FEATURES_PAGE_MAX; !found && i >= 0; i--) {
-      for (int j = 0; j < HCI_FEATURE_BYTES_PER_PAGE; j++) {
-        if (p_dev_rec->feature_pages[i][j] != 0) {
-          found = true;
-          p_dev_rec->num_read_pages = i + 1;
-          break;
-        }
-      }
-    }
-  } else {
-    memset(p_dev_rec->feature_pages, 0, sizeof(p_dev_rec->feature_pages));
-  }
+  memset(p_dev_rec->feature_pages, 0, sizeof(p_dev_rec->feature_pages));
 
   if (p_link_key) {
     VLOG(2) << __func__ << ": BDA: " << bd_addr;