Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 5a258b0d authored by Chen Chen's avatar Chen Chen Committed by Automerger Merge Worker
Browse files

BluetoothMetrics: Do not assert on events from remote device before upload... 

BluetoothMetrics: Do not assert on events from remote device before upload them am: e18e8579 am: 3ac45841

Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/modules/Bluetooth/+/18145083



Change-Id: Id7f47a80fb1beabaa42e1edcffad724924e1343c
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 2d89aa80 3ac45841

system/gd/hci/hci_metrics_logging.cc

+32 −8
Original line number Diff line number Diff line
@@ -652,7 +652,10 @@ void log_classic_pairing_other_hci_event(EventView packet) { 
    }

    case EventCode::REMOTE_OOB_DATA_REQUEST: {

      RemoteOobDataRequestView remote_oob_data_request_view = RemoteOobDataRequestView::Create(std::move(packet));

      ASSERT(remote_oob_data_request_view.IsValid());

      if (!remote_oob_data_request_view.IsValid()) {

        LOG_WARN("remote_oob_data_request_view not valid");

        return;

      }

      address = remote_oob_data_request_view.GetBdAddr();

      break;

    }
@@ -665,7 +668,10 @@ void log_classic_pairing_other_hci_event(EventView packet) { 
    }

    case EventCode::REMOTE_NAME_REQUEST_COMPLETE: {

      RemoteNameRequestCompleteView remote_name_request_complete_view = RemoteNameRequestCompleteView::Create(std::move(packet));

      ASSERT(remote_name_request_complete_view.IsValid());

      if (!remote_name_request_complete_view.IsValid()) {

        LOG_WARN("remote_name_request_complete_view not valid");

        return;

      }

      address = remote_name_request_complete_view.GetBdAddr();

      status = remote_name_request_complete_view.GetStatus();

      break;
@@ -831,14 +837,20 @@ void log_classic_pairing_command_status(std::unique_ptr<CommandView>& command_vi 
    case OpCode::REMOTE_OOB_DATA_REQUEST_REPLY: {

      RemoteOobDataRequestReplyView remote_oob_data_request_reply_view

      = RemoteOobDataRequestReplyView::Create(std::move(security_command_view));

      ASSERT(remote_oob_data_request_reply_view.IsValid());

      if (!remote_oob_data_request_reply_view.IsValid()) {

        LOG_WARN("remote_oob_data_request_reply_view is not valid.");

        return;

      }

      address = remote_oob_data_request_reply_view.GetBdAddr();

      break;

    }

    case OpCode::REMOTE_OOB_DATA_REQUEST_NEGATIVE_REPLY: {

      RemoteOobDataRequestNegativeReplyView remote_oob_data_request_negative_reply_view

      = RemoteOobDataRequestNegativeReplyView::Create(std::move(security_command_view));

      ASSERT(remote_oob_data_request_negative_reply_view.IsValid());

      if (!remote_oob_data_request_negative_reply_view.IsValid()) {

        LOG_WARN("remote_oob_data_request_negative_reply_view is not valid.");

        return;

      }

      address = remote_oob_data_request_negative_reply_view.GetBdAddr();

      break;

    }
@@ -994,19 +1006,31 @@ void log_classic_pairing_command_complete(EventView event_view, std::unique_ptr< 
    }

    case OpCode::REMOTE_OOB_DATA_REQUEST_REPLY: {

      auto remote_oob_data_request_reply_complete_view = RemoteOobDataRequestReplyCompleteView::Create(std::move(command_complete_view));

      ASSERT(remote_oob_data_request_reply_complete_view.IsValid());

      if (!remote_oob_data_request_reply_complete_view.IsValid()) {

        LOG_WARN("remote_oob_data_request_reply_complete_view is not valid.");

        return;

      }

      status = remote_oob_data_request_reply_complete_view.GetStatus();

      auto remote_oob_data_request_reply_view = RemoteOobDataRequestReplyView::Create(std::move(security_command_view));

      ASSERT(remote_oob_data_request_reply_view.IsValid());

      if (!remote_oob_data_request_reply_view.IsValid()) {

        LOG_WARN("remote_oob_data_request_reply_view is not valid.");

        return;

      }

      address = remote_oob_data_request_reply_view.GetBdAddr();

      break;

    }

    case OpCode::REMOTE_OOB_DATA_REQUEST_NEGATIVE_REPLY: {

      auto remote_oob_data_request_negative_reply_complete_view = RemoteOobDataRequestNegativeReplyCompleteView::Create(std::move(command_complete_view));

      ASSERT(remote_oob_data_request_negative_reply_complete_view.IsValid());

      if (!remote_oob_data_request_negative_reply_complete_view.IsValid()) {

        LOG_WARN("remote_oob_data_request_negative_reply_complete_view is not valid.");

        return;

      }

      status = remote_oob_data_request_negative_reply_complete_view.GetStatus();

      auto remote_oob_data_request_negative_reply_view = RemoteOobDataRequestNegativeReplyView::Create(std::move(security_command_view));

      ASSERT(remote_oob_data_request_negative_reply_view.IsValid());

      if (!remote_oob_data_request_negative_reply_view.IsValid()) {

        LOG_WARN("remote_oob_data_request_negative_reply_view is not valid.");

        return;

      }

      address = remote_oob_data_request_negative_reply_view.GetBdAddr();

      break;

    }