diff --git a/interfaces/inner_kits/native_cpp/audio_sink/src/daudio_sink_ipc_callback.cpp b/interfaces/inner_kits/native_cpp/audio_sink/src/daudio_sink_ipc_callback.cpp index b9bdcb1b8b8d9c254d3b4b2c0428812e071f3860..7b5033fb39140f9bf85039986df06827e42c8007 100644 --- a/interfaces/inner_kits/native_cpp/audio_sink/src/daudio_sink_ipc_callback.cpp +++ b/interfaces/inner_kits/native_cpp/audio_sink/src/daudio_sink_ipc_callback.cpp @@ -36,6 +36,7 @@ int32_t DAudioSinkIpcCallback::OnNotifyResourceInfo(const ResourceEventType &typ std::lock_guard resourceLck(privacyResMtx_); auto iter = privacyResCallback_.begin(); if (iter != privacyResCallback_.end()) { + CHECK_NULL_RETURN(*iter, ERR_DH_AUDIO_NULLPTR); ret = (*iter)->OnPrivaceResourceMessage(type, subType, networkId, isSensitive, isSameAccount); } return ret; diff --git a/interfaces/inner_kits/native_cpp/audio_source/src/daudio_hdf_operate.cpp b/interfaces/inner_kits/native_cpp/audio_source/src/daudio_hdf_operate.cpp index 61a78108da9f353e6adf39cbb8b4bbd6978b4017..40c2e4e941b6fad39fa8028cb7ea77cc8220530a 100644 --- a/interfaces/inner_kits/native_cpp/audio_source/src/daudio_hdf_operate.cpp +++ b/interfaces/inner_kits/native_cpp/audio_source/src/daudio_hdf_operate.cpp @@ -281,7 +281,7 @@ int32_t DaudioHdfOperate::MakeFwkDAudioHdfCallback() { std::lock_guard locker(fwkDAudioHdfCallbackMutex_); if (fwkDAudioHdfCallback_ == nullptr) { - fwkDAudioHdfCallback_ = new FwkDAudioHdfCallback(); + fwkDAudioHdfCallback_ = sptr(new FwkDAudioHdfCallback()); if (fwkDAudioHdfCallback_ == nullptr) { return ERR_DH_AUDIO_NULLPTR; } diff --git a/interfaces/inner_kits/native_cpp/test/unittest/audiosinktest/src/daudio_sink_proxy_test.cpp b/interfaces/inner_kits/native_cpp/test/unittest/audiosinktest/src/daudio_sink_proxy_test.cpp index 393ed6c766927ba9dc398c682d5f56a3df6676f3..0d8c48742c18c0962197b42675e78335466e812f 100644 --- a/interfaces/inner_kits/native_cpp/test/unittest/audiosinktest/src/daudio_sink_proxy_test.cpp +++ b/interfaces/inner_kits/native_cpp/test/unittest/audiosinktest/src/daudio_sink_proxy_test.cpp @@ -104,7 +104,7 @@ HWTEST_F(DAudioSinkProxyTest, InitSink_001, TestSize.Level1) { ASSERT_TRUE(dAudioProxy != nullptr); const std::string params = "params"; - auto dAudioSinkIpcCallback = new DAudioSinkIpcCallback(); + auto dAudioSinkIpcCallback = sptr(new DAudioSinkIpcCallback()); int32_t ret = dAudioProxy->InitSink(params, dAudioSinkIpcCallback); EXPECT_EQ(DH_SUCCESS, ret); ret = dAudioProxy->ReleaseSink(); diff --git a/interfaces/inner_kits/native_cpp/test/unittest/audiosourcetest/src/daudio_ipc_callback_test.cpp b/interfaces/inner_kits/native_cpp/test/unittest/audiosourcetest/src/daudio_ipc_callback_test.cpp index 5768c77a0bd6228e4c8ea64e41a76850e738eb76..ad5f3de69c23d7b1059587c098ca2e28622ec00b 100644 --- a/interfaces/inner_kits/native_cpp/test/unittest/audiosourcetest/src/daudio_ipc_callback_test.cpp +++ b/interfaces/inner_kits/native_cpp/test/unittest/audiosourcetest/src/daudio_ipc_callback_test.cpp @@ -28,7 +28,7 @@ void DAudioIpcCallbackTest::TearDownTestCase(void) {} void DAudioIpcCallbackTest::SetUp(void) { - dAudioIpcCallback_ = new DAudioIpcCallback(); + dAudioIpcCallback_ = sptr(new DAudioIpcCallback()); } void DAudioIpcCallbackTest::TearDown(void) diff --git a/interfaces/inner_kits/native_cpp/test/unittest/audiosourcetest/src/daudio_source_handler_test.cpp b/interfaces/inner_kits/native_cpp/test/unittest/audiosourcetest/src/daudio_source_handler_test.cpp index 0e5752b742c2e6e599ad8a2d28a1cf9d7ca082ed..7f41fb1ead6fd728a02e8ce33dcea76980813cbc 100644 --- a/interfaces/inner_kits/native_cpp/test/unittest/audiosourcetest/src/daudio_source_handler_test.cpp +++ b/interfaces/inner_kits/native_cpp/test/unittest/audiosourcetest/src/daudio_source_handler_test.cpp @@ -50,7 +50,7 @@ HWTEST_F(DAudioSourceHandlerTest, RegisterDistributedHardware_001, TestSize.Leve param.sinkVersion = "1"; param.sinkAttrs = "attrs"; std::shared_ptr callback = std::make_shared(); - DAudioSourceHandler::GetInstance().dAudioSourceProxy_ = new MockIDAudioSource(); + DAudioSourceHandler::GetInstance().dAudioSourceProxy_ = sptr(new MockIDAudioSource()); int32_t ret = DAudioSourceHandler::GetInstance().RegisterDistributedHardware(devId, dhId, param, callback); EXPECT_EQ(DH_SUCCESS, ret); std::shared_ptr uncallback = std::make_shared(); @@ -73,7 +73,7 @@ HWTEST_F(DAudioSourceHandlerTest, RegisterDistributedHardware_002, TestSize.Leve param.sinkAttrs = "attrs"; std::shared_ptr callback = std::make_shared(); DAudioSourceHandler::GetInstance().dAudioIpcCallback_ = nullptr; - DAudioSourceHandler::GetInstance().dAudioSourceProxy_ = new MockIDAudioSource(); + DAudioSourceHandler::GetInstance().dAudioSourceProxy_ = sptr(new MockIDAudioSource()); int32_t ret = DAudioSourceHandler::GetInstance().RegisterDistributedHardware(devId, dhId, param, callback); EXPECT_EQ(ERR_DH_AUDIO_NULLPTR, ret); std::shared_ptr uncallback = std::make_shared(); @@ -121,8 +121,8 @@ HWTEST_F(DAudioSourceHandlerTest, RegisterDistributedHardware_004, TestSize.Leve param.sinkVersion = "1"; param.sinkAttrs = "attrs"; std::shared_ptr callback = std::make_shared(); - DAudioSourceHandler::GetInstance().dAudioSourceProxy_ = new MockIDAudioSource(); - DAudioSourceHandler::GetInstance().dAudioIpcCallback_ = new DAudioIpcCallback(); + DAudioSourceHandler::GetInstance().dAudioSourceProxy_ = sptr(new MockIDAudioSource()); + DAudioSourceHandler::GetInstance().dAudioIpcCallback_ = sptr(new DAudioIpcCallback()); int32_t ret = DAudioSourceHandler::GetInstance().RegisterDistributedHardware(devId, dhId, param, callback); EXPECT_EQ(ERR_DH_AUDIO_SA_DEVID_ILLEGAL, ret); std::shared_ptr uncallback = std::make_shared(); @@ -157,7 +157,7 @@ HWTEST_F(DAudioSourceHandlerTest, ReleaseSource_001, TestSize.Level1) */ HWTEST_F(DAudioSourceHandlerTest, ReleaseSource_002, TestSize.Level1) { - DAudioSourceHandler::GetInstance().dAudioSourceProxy_ = new MockIDAudioSource(); + DAudioSourceHandler::GetInstance().dAudioSourceProxy_ = sptr(new MockIDAudioSource()); int32_t ret = DAudioSourceHandler::GetInstance().ReleaseSource(); EXPECT_EQ(DH_SUCCESS, ret); wptr remote = nullptr; @@ -196,7 +196,7 @@ HWTEST_F(DAudioSourceHandlerTest, ConfigDistributedHardware_002, TestSize.Level1 size_t DAUDIO_LEGAL_DEVICE_ID_LEN = 10; std::string devId; devId.resize(DAUDIO_MAX_DEVICE_ID_LEN); - DAudioSourceHandler::GetInstance().dAudioSourceProxy_ = new MockIDAudioSource(); + DAudioSourceHandler::GetInstance().dAudioSourceProxy_ = sptr(new MockIDAudioSource()); int32_t ret = DAudioSourceHandler::GetInstance().ConfigDistributedHardware(devId, dhId, key, value); EXPECT_EQ(ERR_DH_AUDIO_SA_DEVID_ILLEGAL, ret); devId.resize(DAUDIO_LEGAL_DEVICE_ID_LEN); @@ -217,7 +217,7 @@ HWTEST_F(DAudioSourceHandlerTest, ConfigDistributedHardware_003, TestSize.Level1 std::string dhId = "dhId"; std::string key = "key"; std::string value = "value"; - DAudioSourceHandler::GetInstance().dAudioSourceProxy_ = new MockIDAudioSource(); + DAudioSourceHandler::GetInstance().dAudioSourceProxy_ = sptr(new MockIDAudioSource()); int32_t ret = DAudioSourceHandler::GetInstance().ConfigDistributedHardware(devId, dhId, key, value); EXPECT_EQ(DH_SUCCESS, ret); } diff --git a/services/audiocontrol/test/unittest/controlsource/src/daudio_source_dev_ctrl_manager_test.cpp b/services/audiocontrol/test/unittest/controlsource/src/daudio_source_dev_ctrl_manager_test.cpp index a3a63c8611aa349ebe606466f601cd9222ffaae0..fe8a711ab0be4e83727b711f0e693e62404fc979 100644 --- a/services/audiocontrol/test/unittest/controlsource/src/daudio_source_dev_ctrl_manager_test.cpp +++ b/services/audiocontrol/test/unittest/controlsource/src/daudio_source_dev_ctrl_manager_test.cpp @@ -30,7 +30,7 @@ void DAudioSourceDevCtrlMgrTest::TearDownTestCase(void) {} void DAudioSourceDevCtrlMgrTest::SetUp(void) { uint64_t tokenId; - const char** perms = new const char *[2]; + const char* perms[2] = {"ohos.permisson.DISTRIBUTED_DATASYNC", "ohos.permission.DISTRIBUTED_SOFTBUS_CENTER"}; perms[0] = "ohos.permission.DISTRIBUTED_DATASYNC"; perms[1] = "ohos.permission.DISTRIBUTED_SOFTBUS_CENTER"; NativeTokenInfoParams infoInstance = { diff --git a/services/audiohdiproxy/src/daudio_hdi_handler.cpp b/services/audiohdiproxy/src/daudio_hdi_handler.cpp index 50cd997f1082a5a595396a5460cda627ba8e07e6..29fcb3562a1be02501b2a6463d88ab094c534dde 100644 --- a/services/audiohdiproxy/src/daudio_hdi_handler.cpp +++ b/services/audiohdiproxy/src/daudio_hdi_handler.cpp @@ -40,7 +40,7 @@ IMPLEMENT_SINGLE_INSTANCE(DAudioHdiHandler); DAudioHdiHandler::DAudioHdiHandler() { DHLOGD("Distributed audio hdi handler construct."); - audioHdiRecipient_ = new AudioHdiRecipient(); + audioHdiRecipient_ = sptr(new AudioHdiRecipient()); } DAudioHdiHandler::~DAudioHdiHandler() diff --git a/services/audiohdiproxy/test/unittest/daudio_hdi_handler/src/daudio_hdi_handler_test.cpp b/services/audiohdiproxy/test/unittest/daudio_hdi_handler/src/daudio_hdi_handler_test.cpp index 793084380030e899361fba7264a0a030573b962e..4e51346b1f820cbc20fec46a64a7ee75bd72d41a 100644 --- a/services/audiohdiproxy/test/unittest/daudio_hdi_handler/src/daudio_hdi_handler_test.cpp +++ b/services/audiohdiproxy/test/unittest/daudio_hdi_handler/src/daudio_hdi_handler_test.cpp @@ -109,7 +109,7 @@ HWTEST_F(DAudioHdiHandlerTest, NotifyEvent_002, TestSize.Level1) { ASSERT_TRUE(hdiHandler_ != nullptr); EXPECT_EQ(HDF_SUCCESS, hdiHandler_->InitHdiHandler()); - hdiHandler_->audioSrvHdf_ = new MockIDAudioManager(); + hdiHandler_->audioSrvHdf_ = sptr(new MockIDAudioManager()); AudioEvent audioEvent1(AudioEventType::NOTIFY_OPEN_SPEAKER_RESULT, ""); EXPECT_EQ(HDF_SUCCESS, hdiHandler_->NotifyEvent(devId_, dhId_, streamId_, audioEvent1)); AudioEvent audioEvent2(AudioEventType::NOTIFY_CLOSE_SPEAKER_RESULT, "");