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

Commit 2b8c6306 authored by Brian Delwiche's avatar Brian Delwiche Committed by Automerger Merge Worker
Browse files

Merge "[conflict] Merge "Add bounds checks in btif_avrcp_audio_track.cc" into... 

Merge "[conflict] Merge "Add bounds checks in btif_avrcp_audio_track.cc" into tm-dev am: 0b68bd68 am: 52d169b1" into udc-dev-plus-aosp am: be18a381

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



Change-Id: I379b92fc5507d9dd898a1866a28be5165f5ded4e
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents f356d2e5 be18a381

system/btif/src/btif_avrcp_audio_track.cc

+3 −3
Original line number Diff line number Diff line
@@ -221,7 +221,7 @@ static size_t transcodeQ15ToFloat(uint8_t* buffer, size_t length, 
  size_t sampleSize = sampleSizeFor(trackHolder);

  size_t i = 0;

  const float scaledGain = trackHolder->gain * kScaleQ15ToFloat;

  for (; i <= length / sampleSize; i++) {

  for (; i < std::min(trackHolder->bufferLength, length / sampleSize); i++) {

    trackHolder->buffer[i] = ((int16_t*)buffer)[i] * scaledGain;

  }

  return i * sampleSize;
@@ -232,7 +232,7 @@ static size_t transcodeQ23ToFloat(uint8_t* buffer, size_t length, 
  size_t sampleSize = sampleSizeFor(trackHolder);

  size_t i = 0;

  const float scaledGain = trackHolder->gain * kScaleQ23ToFloat;

  for (; i <= length / sampleSize; i++) {

  for (; i < std::min(trackHolder->bufferLength, length / sampleSize); i++) {

    size_t offset = i * sampleSize;

    int32_t sample = *((int32_t*)(buffer + offset - 1)) & 0x00FFFFFF;

    trackHolder->buffer[i] = sample * scaledGain;
@@ -245,7 +245,7 @@ static size_t transcodeQ31ToFloat(uint8_t* buffer, size_t length, 
  size_t sampleSize = sampleSizeFor(trackHolder);

  size_t i = 0;

  const float scaledGain = trackHolder->gain * kScaleQ31ToFloat;

  for (; i <= length / sampleSize; i++) {

  for (; i < std::min(trackHolder->bufferLength, length / sampleSize); i++) {

    trackHolder->buffer[i] = ((int32_t*)buffer)[i] * scaledGain;

  }

  return i * sampleSize;

system/btif/test/btif_avrcp_audio_track_test.cc

+1 −1
Original line number Diff line number Diff line
@@ -84,7 +84,7 @@ TEST_F(BtifAvrcpAudioTrackTest, 
  for (size_t index = 0; index < bufferLength; ++index) {

    data[index] = index;

  }

  BtifAvrcpAudioTrackWriteData(trackHolder, data, bufferLength - 1);

  BtifAvrcpAudioTrackWriteData(trackHolder, data, bufferLength);

  const int16_t* dataInt = (int16_t*)data;

  for (size_t index = 0; index < bufferLength / sampleSize; ++index) {

    const float expected = dataInt[index] * scaleQ15ToFloat * gainValue;