diff --git a/frameworks/native/session/test/unittest/avsession_dumper_test.cpp b/frameworks/native/session/test/unittest/avsession_dumper_test.cpp index 5ce9ee9decd8bb262d2a2523625d3f3d00772b88..253495e27452b107105db6fa4c857ff7c21daa27 100644 --- a/frameworks/native/session/test/unittest/avsession_dumper_test.cpp +++ b/frameworks/native/session/test/unittest/avsession_dumper_test.cpp @@ -536,8 +536,9 @@ HWTEST_F(AVSessionDumperTest, SetDeviceInfo001, TestSize.Level1) */ HWTEST_F(AVSessionDumperTest, GetAudioDescriptorByDeviceId001, TestSize.Level1) { - std::vector> castAudioDescriptors; - sptr des = new AudioStandard::AudioDeviceDescriptor(); + std::vector> castAudioDescriptors; + std::shared_ptr des = + std::make_shared(); AudioStandard::AudioDeviceDescriptor res; des->deviceId_ = 12; castAudioDescriptors.push_back(des); @@ -553,8 +554,9 @@ HWTEST_F(AVSessionDumperTest, GetAudioDescriptorByDeviceId001, TestSize.Level1) */ HWTEST_F(AVSessionDumperTest, GetAudioDescriptorByDeviceId002, TestSize.Level1) { - std::vector> castAudioDescriptors; - sptr des = new AudioStandard::AudioDeviceDescriptor(); + std::vector> castAudioDescriptors; + std::shared_ptr des = + std::make_shared(); AudioStandard::AudioDeviceDescriptor res; des->deviceId_ = 11; castAudioDescriptors.push_back(des); diff --git a/services/session/adapter/audio_adapter.cpp b/services/session/adapter/audio_adapter.cpp index 7e77a1716064670ee5a72e186ab7edd8931db6da..20a3dc28adb912358f1ee484d64bd35cfaf0e4af 100644 --- a/services/session/adapter/audio_adapter.cpp +++ b/services/session/adapter/audio_adapter.cpp @@ -42,8 +42,8 @@ AudioAdapter::~AudioAdapter() void AudioAdapter::Init() { SLOGI("register audio renderer event listener"); - auto ret = AudioStandard::AudioStreamManager::GetInstance()->RegisterAudioRendererEventListener(getpid(), - shared_from_this()); + auto ret = AudioStandard::AudioStreamManager::GetInstance()->RegisterAudioRendererEventListener( + getpid(), shared_from_this()); if (ret != 0) { SLOGE("register audio renderer event listener failed"); } @@ -69,7 +69,8 @@ void AudioAdapter::AddDeviceChangeListener(const PreferOutputDeviceChangeListene int32_t AudioAdapter::MuteAudioStream(int32_t uid, int32_t pid) { std::vector> audioRendererChangeInfo; - auto ret = AudioStandard::AudioStreamManager::GetInstance()->GetCurrentRendererChangeInfos(audioRendererChangeInfo); + auto ret = + AudioStandard::AudioStreamManager::GetInstance()->GetCurrentRendererChangeInfos(audioRendererChangeInfo); if (ret != 0) { SLOGE("get renderer state failed"); return AVSESSION_ERROR; @@ -97,7 +98,8 @@ int32_t AudioAdapter::MuteAudioStream(int32_t uid, int32_t pid) int32_t AudioAdapter::UnMuteAudioStream(int32_t uid) { std::vector> audioRendererChangeInfo; - auto ret = AudioStandard::AudioStreamManager::GetInstance()->GetCurrentRendererChangeInfos(audioRendererChangeInfo); + auto ret = + AudioStandard::AudioStreamManager::GetInstance()->GetCurrentRendererChangeInfos(audioRendererChangeInfo); if (ret != 0) { SLOGE("get renderer state failed"); return AVSESSION_ERROR; @@ -168,7 +170,7 @@ void AudioAdapter::OnDeviceChange(const DeviceChangeAction& deviceChangeAction) { } -void AudioAdapter::OnPreferredOutputDeviceUpdated(const std::vector> &desc) +void AudioAdapter::OnPreferredOutputDeviceUpdated(const std::vector> &desc) { for (const auto& listener : deviceChangeListeners_) { if (listener) { @@ -180,7 +182,8 @@ void AudioAdapter::OnPreferredOutputDeviceUpdated(const std::vector> audioRendererChangeInfo; - auto ret = AudioStandard::AudioStreamManager::GetInstance()->GetCurrentRendererChangeInfos(audioRendererChangeInfo); + auto ret = + AudioStandard::AudioStreamManager::GetInstance()->GetCurrentRendererChangeInfos(audioRendererChangeInfo); if (ret != 0) { SLOGE("get renderer state failed"); return false; diff --git a/services/session/adapter/audio_adapter.h b/services/session/adapter/audio_adapter.h index 47199f7ef654cf714b1a574f85a957cd2845eadd..e792cdcd1b124613a2b0547b4ceba8c2780a0d1e 100644 --- a/services/session/adapter/audio_adapter.h +++ b/services/session/adapter/audio_adapter.h @@ -34,7 +34,8 @@ class AudioAdapter : public AudioStandard::AudioRendererStateChangeCallback, public: using StateListener = std::function; using DeviceChangeListener = std::function; - using PreferOutputDeviceChangeListener = std::function> &desc)>; + using PreferOutputDeviceChangeListener = + std::function> &desc)>; static AudioAdapter& GetInstance(); AudioAdapter(); @@ -60,7 +61,7 @@ public: void OnDeviceChange(const DeviceChangeAction& deviceChangeAction) override; - void OnPreferredOutputDeviceUpdated(const std::vector> &desc) override; + void OnPreferredOutputDeviceUpdated(const std::vector> &desc) override; bool GetRendererRunning(int32_t uid); diff --git a/services/session/server/audio_device_manager.cpp b/services/session/server/audio_device_manager.cpp index d2a329ada324a7ffb7c035fbbf1d5ddad27f559e..a909b83be9a7fa516e8da4b836fe6aed9005516d 100644 --- a/services/session/server/audio_device_manager.cpp +++ b/services/session/server/audio_device_manager.cpp @@ -27,7 +27,7 @@ bool AudioDeviceManager::GetVehicleA2dpConnectState() { AudioStandard::AudioSystemManager *audioSystemManager = AudioStandard::AudioSystemManager::GetInstance(); - std::vector> devices = + std::vector> devices = audioSystemManager->GetActiveOutputDeviceDescriptors(); bool isConnected = false; for (auto& device : devices) { @@ -100,7 +100,7 @@ void DeviceChangeCallback::OnDeviceChange(const AudioStandard::DeviceChangeActio return; } if (AudioStandard::DeviceChangeType::DISCONNECT == deviceChangeAction.type) { - std::vector> deviceDescriptors = + std::vector> deviceDescriptors = deviceChangeAction.deviceDescriptors; for (auto &device : deviceDescriptors) { if (device != nullptr && diff --git a/services/session/server/avsession_service.cpp b/services/session/server/avsession_service.cpp index dd74670b9bbbfc36be4b247f2aa526527eac929f..1ac14f2ecafd199bacb3f6815d7db57fa598e3f0 100644 --- a/services/session/server/avsession_service.cpp +++ b/services/session/server/avsession_service.cpp @@ -716,7 +716,8 @@ void AVSessionService::InitAudio() AudioAdapter::GetInstance().AddStreamRendererStateListener([this] (const AudioRendererChangeInfos& infos) { OutputDeviceChangeListener(infos); }); - AudioAdapter::GetInstance().AddDeviceChangeListener([this] (const std::vector> &desc) { + AudioAdapter::GetInstance().AddDeviceChangeListener( + [this] (const std::vector> &desc) { HandleDeviceChange(desc); }); } @@ -2227,8 +2228,9 @@ void AVSessionService::SetDeviceInfo(const std::vectorSetOutputDevice(outputDeviceInfo); } -bool AVSessionService::GetAudioDescriptorByDeviceId(const std::vector>& - descriptors, const std::string& deviceId, +bool AVSessionService::GetAudioDescriptorByDeviceId( + const std::vector>& descriptors, + const std::string& deviceId, AudioStandard::AudioDeviceDescriptor& audioDescriptor) { for (const auto& descriptor : descriptors) { @@ -2271,8 +2273,8 @@ int32_t AVSessionService::SelectOutputDevice(const int32_t uid, const AudioDevic audioFilter->rendererInfo.contentType = ContentType::CONTENT_TYPE_MUSIC; audioFilter->rendererInfo.streamUsage = StreamUsage::STREAM_USAGE_MEDIA; - std::vector> audioDescriptor; - auto audioDeviceDescriptor = new(std::nothrow) AudioDeviceDescriptor(descriptor); + std::vector> audioDescriptor; + auto audioDeviceDescriptor = std::make_shared(descriptor); CHECK_AND_RETURN_RET_LOG(audioDeviceDescriptor != nullptr, ERR_NO_MEMORY, "audioDeviceDescriptor is nullptr"); audioDescriptor.push_back(audioDeviceDescriptor); SLOGI("select the device %{public}s role is %{public}d, networkId is %{public}.6s", @@ -2896,7 +2898,8 @@ void AVSessionService::NotifyDeviceChange() // LCOV_EXCL_STOP // LCOV_EXCL_START -void AVSessionService::HandleDeviceChange(const std::vector> &desc) +void AVSessionService::HandleDeviceChange( + const std::vector> &desc) { for (auto &audioDeviceDescriptor : desc) { if (audioDeviceDescriptor->deviceType_ == AudioStandard::DEVICE_TYPE_WIRED_HEADSET || diff --git a/services/session/server/avsession_service.h b/services/session/server/avsession_service.h index 6fbf4065f93f76163a3204f4aa616c5de44d19fd..a42fa6c1f20ea8f1e3b27c9c1a3adbc20c50993b 100644 --- a/services/session/server/avsession_service.h +++ b/services/session/server/avsession_service.h @@ -341,7 +341,7 @@ private: void HandleFocusSession(const FocusSessionStrategy::FocusSessionChangeInfo& info); - void HandleDeviceChange(const std::vector> &desc); + void HandleDeviceChange(const std::vector> &desc); __attribute__((no_sanitize("cfi"))) std::shared_ptr GetService( const std::string& deviceId); @@ -383,9 +383,10 @@ private: int32_t GetAudioDescriptor(const std::string deviceId, std::vector& audioDeviceDescriptors); - bool GetAudioDescriptorByDeviceId(const std::vector>& descriptors, - const std::string& deviceId, - AudioStandard::AudioDeviceDescriptor& audioDescriptor); + bool GetAudioDescriptorByDeviceId( + const std::vector>& descriptors, + const std::string& deviceId, + AudioStandard::AudioDeviceDescriptor& audioDescriptor); void GetDeviceInfo(const sptr& session, const std::vector& descriptors, diff --git a/services/session/server/test/audio_adapter_test.cpp b/services/session/server/test/audio_adapter_test.cpp index 951f364b3470bd0044c20157cef39cfdbabaac22..f98d7e1754528a2ee1a858a79121ea40385dfcdd 100644 --- a/services/session/server/test/audio_adapter_test.cpp +++ b/services/session/server/test/audio_adapter_test.cpp @@ -248,7 +248,7 @@ static HWTEST(AudioAdapterTest, OnDeviceChange001, TestSize.Level1) SLOGD("PauseAudioStream001 begin!"); bool ret = false; AudioAdapter::GetInstance().AddDeviceChangeListener( - [&ret] (const std::vector> &desc) { + [&ret] (const std::vector> &desc) { ret = desc.empty(); }); AudioAdapter::GetInstance().AddDeviceChangeListener(nullptr); diff --git a/services/session/server/test/avsession_service_supplement_test.cpp b/services/session/server/test/avsession_service_supplement_test.cpp index 609a6c11e918d03bc5d25ae107996150d7b02964..23a261d802635121ad3cb302c681c3c9820ec3c0 100644 --- a/services/session/server/test/avsession_service_supplement_test.cpp +++ b/services/session/server/test/avsession_service_supplement_test.cpp @@ -135,8 +135,9 @@ static HWTEST_F(AVSessionServiceSupplementTest, SetDeviceInfo001, TestSize.Level static HWTEST_F(AVSessionServiceSupplementTest, GetAudioDescriptorByDeviceId001, TestSize.Level1) { SLOGI("GetAudioDescriptorByDeviceId001 begin!"); - std::vector> descriptors; - sptr descriptor = new AudioStandard::AudioDeviceDescriptor(); + std::vector> descriptors; + std::shared_ptr descriptor = + std::make_shared(); descriptors.push_back(descriptor); std::string deviceId = "0"; AudioStandard::AudioDeviceDescriptor audioDescriptor; @@ -153,8 +154,9 @@ static HWTEST_F(AVSessionServiceSupplementTest, GetAudioDescriptorByDeviceId001, static HWTEST_F(AVSessionServiceSupplementTest, GetAudioDescriptorByDeviceId002, TestSize.Level1) { SLOGI("GetAudioDescriptorByDeviceId002 begin!"); - std::vector> descriptors; - sptr descriptor = new AudioStandard::AudioDeviceDescriptor(); + std::vector> descriptors; + std::shared_ptr descriptor = + std::make_shared(); descriptors.push_back(descriptor); std::string deviceId = "1"; AudioStandard::AudioDeviceDescriptor audioDescriptor; diff --git a/services/session/server/test/avsession_service_test.cpp b/services/session/server/test/avsession_service_test.cpp index e2cbaee5bd960c4c0db99db3894ca9583ab23a90..a0ef3962c141ba2423fe42f88f84f3a4593a5129 100644 --- a/services/session/server/test/avsession_service_test.cpp +++ b/services/session/server/test/avsession_service_test.cpp @@ -850,8 +850,8 @@ static HWTEST_F(AVSessionServiceTest, HandleDeviceChange001, TestSize.Level1) { SLOGI("HandleDeviceChange001 begin!"); DeviceChangeAction deviceChange; - std::vector> desc; - OHOS::sptr descriptor = new(std::nothrow) AudioDeviceDescriptor(); + std::vector> desc; + std::shared_ptr descriptor = std::make_shared(); descriptor->deviceType_ = OHOS::AudioStandard::DEVICE_TYPE_WIRED_HEADSET; deviceChange.type = static_cast(0); deviceChange.flag = static_cast(0); @@ -865,8 +865,8 @@ static HWTEST_F(AVSessionServiceTest, HandleDeviceChange002, TestSize.Level1) { SLOGI("HandleDeviceChange002 begin!"); DeviceChangeAction deviceChange; - std::vector> audioDeviceDescriptors; - OHOS::sptr descriptor = new(std::nothrow) AudioDeviceDescriptor(); + std::vector> audioDeviceDescriptors; + std::shared_ptr descriptor = std::make_shared(); descriptor->deviceType_ = OHOS::AudioStandard::DEVICE_TYPE_WIRED_HEADSET; deviceChange.type = static_cast(0); deviceChange.flag = static_cast(0); diff --git a/services/session/server/test/fuzztest/avsessionservice_fuzzer/avsessionservice_fuzzer.cpp b/services/session/server/test/fuzztest/avsessionservice_fuzzer/avsessionservice_fuzzer.cpp index 5de0e2e05cbea89498047ee04d67f9fb1fa482f3..e40a981ddf667119c0c9586a49edd99eadbccd67 100644 --- a/services/session/server/test/fuzztest/avsessionservice_fuzzer/avsessionservice_fuzzer.cpp +++ b/services/session/server/test/fuzztest/avsessionservice_fuzzer/avsessionservice_fuzzer.cpp @@ -711,8 +711,8 @@ void ConvertKeyCodeToCommand001(const uint8_t* data, size_t size) void HandleDeviceChange001(const uint8_t* data, size_t size) { SLOGI("HandleDeviceChange001 begin!"); - std::vector> audioDeviceDescriptors; - OHOS::sptr descriptor = new(std::nothrow) AudioDeviceDescriptor(); + std::vector> audioDeviceDescriptors; + std::shared_ptr descriptor = std::make_shared(); descriptor->deviceType_ = OHOS::AudioStandard::DEVICE_TYPE_WIRED_HEADSET; audioDeviceDescriptors.push_back(descriptor); avsessionService_->HandleDeviceChange(audioDeviceDescriptors); @@ -723,8 +723,8 @@ void HandleDeviceChange002(const uint8_t* data, size_t size) { SLOGI("HandleDeviceChange002 begin!"); DeviceChangeAction deviceChange; - std::vector> audioDeviceDescriptors; - OHOS::sptr descriptor = new(std::nothrow) AudioDeviceDescriptor(); + std::vector> audioDeviceDescriptors; + std::shared_ptr descriptor = std::make_shared(); descriptor->deviceType_ = OHOS::AudioStandard::DEVICE_TYPE_WIRED_HEADSET; deviceChange.type = static_cast(0); deviceChange.flag = static_cast(0);