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

Commit e01b0f7c authored by Grzegorz Kołodziejczyk's avatar Grzegorz Kołodziejczyk Committed by Jakub Pawlowski
Browse files

Reset data queue pointer in negative config cases 

Data queue should be set to nullptr if pcm configuration is invalid.
Otherwise previously set queue would be cosidered during check.

Tag: #stability
Test: vts-tradefed run vts -m VtsHalBluetoothAudioV2_1TargetTest
Sponsor: jpawlowski@
Fixes: 173353147
Bug: 173353147
Change-Id: I10ee061e1c75f73410a9d0a0b1eaace0e536addd
parent d47c62b6

bluetooth/audio/2.1/vts/functional/VtsHalBluetoothAudioV2_1TargetTest.cpp

+2 −0
Original line number Diff line number Diff line
@@ -1043,6 +1043,7 @@ TEST_P(BluetoothAudioProviderLeAudioOutputSoftwareHidlTest, 
    } else {

      EXPECT_EQ(status, BluetoothAudioStatus::UNSUPPORTED_CODEC_CONFIGURATION);

      EXPECT_FALSE(dataMQ.isHandleValid());

      tempDataMQ.reset(nullptr);

    }

  };

  android::hardware::bluetooth::audio::V2_1::AudioConfiguration audio_config =
@@ -1132,6 +1133,7 @@ TEST_P(BluetoothAudioProviderLeAudioInputSoftwareHidlTest, 
    } else {

      EXPECT_EQ(status, BluetoothAudioStatus::UNSUPPORTED_CODEC_CONFIGURATION);

      EXPECT_FALSE(dataMQ.isHandleValid());

      tempDataMQ.reset(nullptr);

    }

  };

  android::hardware::bluetooth::audio::V2_1::AudioConfiguration audio_config =