gd: hci: Ignore unexpected status events

For some reason, on some old devices, the controller will report a
remote to support SNIFF_SUBRATING even when it does not. Just ignore the
error here (the status event comes from the failure response).

patches:
F DEBUG   : pid: 3493, tid: 3527, name: bt_stack_manage  >>> com.android.bluetooth <<<
F DEBUG   : Abort message: 'assertion 'command_queue_.front().waiting_for_status_ == is_status' failed - 0x200d (LE_CREATE_CONNECTION) was not expecting complete event'

Co-authored-by: Peter Cai <[email protected]>
Signed-off-by: fukiame <[email protected]>
Signed-off-by: Wahid Khan <[email protected]>

Author: 2 people

system/gd/hci/hci_layer.cc

@@ -218,14 +218,13 @@ struct HciLayer::impl {
       command_queue_.front().GetCallback<CommandCompleteView>()->Invoke(
           std::move(command_complete_view));
     } else {
-      ASSERT_LOG(
-          command_queue_.front().waiting_for_status_ == is_status,
-          "0x%02hx (%s) was not expecting %s event",
-          op_code,
-          OpCodeText(op_code).c_str(),
-          logging_id.c_str());
-
-      command_queue_.front().GetCallback<TResponse>()->Invoke(std::move(response_view));
+      if (command_queue_.front().waiting_for_status_ == is_status) {
+        command_queue_.front().GetCallback<TResponse>()->Invoke(std::move(response_view));
+      } else {
+        CommandCompleteView command_complete_view = CommandCompleteView::Create(
+            EventView::Create(PacketView<kLittleEndian>(std::make_shared<std::vector<uint8_t>>(std::vector<uint8_t>()))));
+        command_queue_.front().GetCallback<CommandCompleteView>()->Invoke(std::move(command_complete_view));
+      }
     }
 
 #ifdef TARGET_FLOSS