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

Commit 27386e59 authored by Hansong Zhang's avatar Hansong Zhang Committed by android-build-merger
Browse files

Merge "AVRCP: Verify SetBrowsedPlayer player ID"

am: b868620f

Change-Id: I1bb359ff06ed6414ccc0c6440a2b9000caba871b
parents 65484a49 b868620f
Loading
Loading
Loading
Loading
+7 −2
Original line number Diff line number Diff line
@@ -1009,8 +1009,13 @@ void Device::SetBrowsedPlayerResponse(
  DEVICE_VLOG(2) << __func__ << ": success=" << success << " root_id=\""
                 << root_id << "\" num_items=" << num_items;

  // TODO (apanicke): Check success. Right now this is ok since it will
  // always succeed since we only have one player in the media layer.
  if (!success) {
    auto response = SetBrowsedPlayerResponseBuilder::MakeBuilder(
        Status::INVALID_PLAYER_ID, 0x0000, num_items, 0, "");
    send_message(label, true, std::move(response));
    return;
  }

  curr_browsed_player_id_ = pkt->GetPlayerId();

  // Clear the path and push the new root.