diff --git a/interfaces/inner_kits/native_cpp/audio_sink/src/daudio_sink_ipc_callback_stub.cpp b/interfaces/inner_kits/native_cpp/audio_sink/src/daudio_sink_ipc_callback_stub.cpp index a0f6258c4d77f03efee1a072129bf067dece6a64..e794b043fd15ce2506893a95c268adc151a6cafb 100644 --- a/interfaces/inner_kits/native_cpp/audio_sink/src/daudio_sink_ipc_callback_stub.cpp +++ b/interfaces/inner_kits/native_cpp/audio_sink/src/daudio_sink_ipc_callback_stub.cpp @@ -23,7 +23,7 @@ namespace OHOS { namespace DistributedHardware { -DAudioSinkIpcCallbackStub::DAudioSinkIpcCallbackStub() +DAudioSinkIpcCallbackStub::DAudioSinkIpcCallbackStub() : IRemoteStub(true) { memberFuncMap_[NOTIFY_RESOURCEINFO] = &DAudioSinkIpcCallbackStub::OnNotifyResourceInfoInner; } diff --git a/interfaces/inner_kits/native_cpp/audio_source/src/daudio_ipc_callback_stub.cpp b/interfaces/inner_kits/native_cpp/audio_source/src/daudio_ipc_callback_stub.cpp index 68f950c24112227dedde3902399cc3b9afa04c59..35e183de57c9125e1346bea2206e04d0b25611db 100644 --- a/interfaces/inner_kits/native_cpp/audio_source/src/daudio_ipc_callback_stub.cpp +++ b/interfaces/inner_kits/native_cpp/audio_source/src/daudio_ipc_callback_stub.cpp @@ -23,7 +23,7 @@ namespace OHOS { namespace DistributedHardware { -DAudioIpcCallbackStub::DAudioIpcCallbackStub() +DAudioIpcCallbackStub::DAudioIpcCallbackStub() : IRemoteStub(true) { memberFuncMap_[NOTIFY_REGRESULT] = &DAudioIpcCallbackStub::OnNotifyRegResultInner; memberFuncMap_[NOTIFY_UNREGRESULT] = &DAudioIpcCallbackStub::OnNotifyUnregResultInner; 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 27df7a7e8528ebeb9764d0ac44d184eeccb03bf0..9d6d3dfe874f68e08ce283cb1bd79885c8c6a0ab 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 @@ -82,14 +82,23 @@ HWTEST_F(DAudioIpcCallbackTest, OnNotifyRegResult_002, TestSize.Level1) HWTEST_F(DAudioIpcCallbackTest, OnNotifyRegResult_003, TestSize.Level1) { size_t DAUDIO_MAX_DEVICE_ID_LEN = 101; + size_t DAUDIO_LEGAL_DEVICE_ID_LEN = 10; std::string devId ; devId.resize(DAUDIO_MAX_DEVICE_ID_LEN); - const std::string dhId = "dhId"; - const std::string reqId = "reqId"; + std::string dhId = "dhId"; + std::string reqId = "reqId"; int32_t status = 0; const std::string data = "data"; int32_t ret = dAudioIpcCallback_->OnNotifyRegResult(devId, dhId, reqId, status, data); EXPECT_EQ(ERR_DH_AUDIO_SA_DEVID_ILLEGAL, ret); + devId.resize(DAUDIO_LEGAL_DEVICE_ID_LEN); + dhId.resize(DAUDIO_MAX_DEVICE_ID_LEN); + ret = dAudioIpcCallback_->OnNotifyRegResult(devId, dhId, reqId, status, data); + EXPECT_EQ(ERR_DH_AUDIO_SA_DEVID_ILLEGAL, ret); + dhId.resize(DAUDIO_LEGAL_DEVICE_ID_LEN); + reqId.resize(DAUDIO_MAX_DEVICE_ID_LEN); + ret = dAudioIpcCallback_->OnNotifyRegResult(devId, dhId, reqId, status, data); + EXPECT_EQ(ERR_DH_AUDIO_SA_DEVID_ILLEGAL, ret); } /** diff --git a/services/audiomanager/servicesink/src/daudio_sink_stub.cpp b/services/audiomanager/servicesink/src/daudio_sink_stub.cpp index 6f4a4cb470cd76d06c1889e505e49bf14316a576..3b777504661143ea5f6721203cefd3dab3d7e2d9 100644 --- a/services/audiomanager/servicesink/src/daudio_sink_stub.cpp +++ b/services/audiomanager/servicesink/src/daudio_sink_stub.cpp @@ -30,7 +30,7 @@ namespace OHOS { namespace DistributedHardware { -DAudioSinkStub::DAudioSinkStub() +DAudioSinkStub::DAudioSinkStub() : IRemoteStub(true) { DHLOGD("Distributed audio sink stub constructed."); memberFuncMap_[static_cast(IDAudioSinkInterfaceCode::INIT_SINK)] = diff --git a/services/audiomanager/servicesource/src/daudio_source_stub.cpp b/services/audiomanager/servicesource/src/daudio_source_stub.cpp index 85b9083ce885e1361e25f271c82102c79166fe73..d00be43ac442f0aeb68964d7fd4c35a2274821bc 100644 --- a/services/audiomanager/servicesource/src/daudio_source_stub.cpp +++ b/services/audiomanager/servicesource/src/daudio_source_stub.cpp @@ -30,7 +30,7 @@ namespace OHOS { namespace DistributedHardware { -DAudioSourceStub::DAudioSourceStub() +DAudioSourceStub::DAudioSourceStub() : IRemoteStub(true) { memberFuncMap_[static_cast(IDAudioSourceInterfaceCode::INIT_SOURCE)] = &DAudioSourceStub::InitSourceInner; diff --git a/services/audiomanager/test/unittest/servicesource/src/daudio_source_service_test.cpp b/services/audiomanager/test/unittest/servicesource/src/daudio_source_service_test.cpp index 21ff2f13f99fc2ca87956123466026a2d0e7b27b..92947a23e56846bf2d5bea817f1d168221c6c5e8 100644 --- a/services/audiomanager/test/unittest/servicesource/src/daudio_source_service_test.cpp +++ b/services/audiomanager/test/unittest/servicesource/src/daudio_source_service_test.cpp @@ -104,6 +104,9 @@ HWTEST_F(DAudioSourceServiceTest, Dump_001, TestSize.Level1) int32_t fd = 1; std::vector args; EXPECT_EQ(DH_SUCCESS, sourceSrv_->Dump(fd, args)); + std::u16string order = u"--sourceDevId"; + args.push_back(order); + EXPECT_EQ(ERR_DH_AUDIO_BAD_VALUE, sourceSrv_->Dump(fd, args)); } } // DistributedHardware } // OHOS