diff --git a/interfaces/innerkits/native_cpp/screen_sink/src/dscreen_sink_handler.cpp b/interfaces/innerkits/native_cpp/screen_sink/src/dscreen_sink_handler.cpp index 725bd3041967a06c72878ad13f951576a5bfc923..91fbe8d3dd1af7806428052b9de87f998a1ab903 100644 --- a/interfaces/innerkits/native_cpp/screen_sink/src/dscreen_sink_handler.cpp +++ b/interfaces/innerkits/native_cpp/screen_sink/src/dscreen_sink_handler.cpp @@ -67,7 +67,7 @@ int32_t DScreenSinkHandler::InitSink(const std::string ¶ms) "dscreen sink get samgr failed."); return ERR_DH_SCREEN_SA_GET_SAMGR_FAIL; } - sptr loadCallback = new DScreenSinkLoadCallback(params); + sptr loadCallback(new DScreenSinkLoadCallback(params)); if (loadCallback == nullptr) { DHLOGE("loadCallback is nullptr."); ReportSaFail(DSCREEN_INIT_FAIL, ERR_DH_SCREEN_SA_GET_SAMGR_FAIL, DISTRIBUTED_HARDWARE_SCREEN_SINK_SA_ID, diff --git a/interfaces/innerkits/native_cpp/screen_source/src/dscreen_source_handler.cpp b/interfaces/innerkits/native_cpp/screen_source/src/dscreen_source_handler.cpp index 7f2da12d8bd6cbe5f6255b382969b5fc233c911d..4e6cfc6cffeb065caa41b42b5a7ad92735ccb7d7 100644 --- a/interfaces/innerkits/native_cpp/screen_source/src/dscreen_source_handler.cpp +++ b/interfaces/innerkits/native_cpp/screen_source/src/dscreen_source_handler.cpp @@ -68,7 +68,7 @@ int32_t DScreenSourceHandler::InitSource(const std::string ¶ms) "dscreen source get samgr failed."); return ERR_DH_SCREEN_SA_GET_SAMGR_FAIL; } - sptr loadCallback = new DScreenSourceLoadCallback(params); + sptr loadCallback(new DScreenSourceLoadCallback(params)); StartTrace(DSCREEN_HITRACE_LABEL, DSCREEN_SOURCE_LOAD_SYSTEM_ABILITY_START); int32_t ret = samgr->LoadSystemAbility(DISTRIBUTED_HARDWARE_SCREEN_SOURCE_SA_ID, loadCallback); if (ret != ERR_OK) { diff --git a/interfaces/innerkits/native_cpp/test/fuzztest/dscreen_source_callback/callbackonnotifyregresult_fuzzer/callbackonnotifyregresult_fuzzer.cpp b/interfaces/innerkits/native_cpp/test/fuzztest/dscreen_source_callback/callbackonnotifyregresult_fuzzer/callbackonnotifyregresult_fuzzer.cpp index 540a86c10e192e901b2770572b29fa882fc240ca..03e3bb3b81aa31208b04078b3afdedae586350a8 100644 --- a/interfaces/innerkits/native_cpp/test/fuzztest/dscreen_source_callback/callbackonnotifyregresult_fuzzer/callbackonnotifyregresult_fuzzer.cpp +++ b/interfaces/innerkits/native_cpp/test/fuzztest/dscreen_source_callback/callbackonnotifyregresult_fuzzer/callbackonnotifyregresult_fuzzer.cpp @@ -39,7 +39,7 @@ void OnNotifyRegResultFuzzTest(const uint8_t* data, size_t size) std::string dataStr(reinterpret_cast(data), size); std::shared_ptr callback = std::make_shared(); - sptr dScreenSourceCallback = new (std::nothrow) DScreenSourceCallback(); + sptr dScreenSourceCallback(new (std::nothrow) DScreenSourceCallback()); dScreenSourceCallback->PushRegRegisterCallback(reqId, callback); dScreenSourceCallback->OnNotifyRegResult(devId, dhId, reqId, status, dataStr); } diff --git a/interfaces/innerkits/native_cpp/test/fuzztest/dscreen_source_callback/callbackonnotifyunregresult_fuzzer/callbackonnotifyunregresult_fuzzer.cpp b/interfaces/innerkits/native_cpp/test/fuzztest/dscreen_source_callback/callbackonnotifyunregresult_fuzzer/callbackonnotifyunregresult_fuzzer.cpp index 1f1bea6f20b2cb418972b59ac19ecd543c214ed3..7fe04bcd4a7b604d377e46f58c4be4393ec9b0a7 100644 --- a/interfaces/innerkits/native_cpp/test/fuzztest/dscreen_source_callback/callbackonnotifyunregresult_fuzzer/callbackonnotifyunregresult_fuzzer.cpp +++ b/interfaces/innerkits/native_cpp/test/fuzztest/dscreen_source_callback/callbackonnotifyunregresult_fuzzer/callbackonnotifyunregresult_fuzzer.cpp @@ -39,7 +39,7 @@ void OnNotifyUnregResultFuzzTest(const uint8_t* data, size_t size) std::string dataStr(reinterpret_cast(data), size); std::shared_ptr uncallback = std::make_shared(); - sptr dScreenSourceCallback = new (std::nothrow) DScreenSourceCallback(); + sptr dScreenSourceCallback(new (std::nothrow) DScreenSourceCallback()); dScreenSourceCallback->PushUnregisterCallback(reqId, uncallback); dScreenSourceCallback->OnNotifyUnregResult(devId, dhId, reqId, status, dataStr); diff --git a/interfaces/innerkits/native_cpp/test/fuzztest/dscreen_source_callback/callbackonremoterequest_fuzzer/callbackonremoterequest_fuzzer.cpp b/interfaces/innerkits/native_cpp/test/fuzztest/dscreen_source_callback/callbackonremoterequest_fuzzer/callbackonremoterequest_fuzzer.cpp index 7d84b784f2a9ad0771bb6845817f926bfce28426..396cb445cd1577831f5c2d8646486c7003c06293 100644 --- a/interfaces/innerkits/native_cpp/test/fuzztest/dscreen_source_callback/callbackonremoterequest_fuzzer/callbackonremoterequest_fuzzer.cpp +++ b/interfaces/innerkits/native_cpp/test/fuzztest/dscreen_source_callback/callbackonremoterequest_fuzzer/callbackonremoterequest_fuzzer.cpp @@ -50,7 +50,7 @@ void CallbackOnRemoteRequestFuzzTest(const uint8_t* data, size_t size) pdata.WriteString(reqId); pdata.WriteString(dataStr); - sptr dScreenSourceCallback = new (std::nothrow) DScreenSourceCallback(); + sptr dScreenSourceCallback(new (std::nothrow) DScreenSourceCallback()); dScreenSourceCallback->OnRemoteRequest(code, pdata, reply, option); } } diff --git a/interfaces/innerkits/native_cpp/test/fuzztest/dscreen_source_proxy/proxyinitsource_fuzzer/initsource_fuzzer.cpp b/interfaces/innerkits/native_cpp/test/fuzztest/dscreen_source_proxy/proxyinitsource_fuzzer/initsource_fuzzer.cpp index aaa4ae91bc71fd37850ea905557c12debf4c1474..ab58c463703a109cc2ed8e71abf4f6e50a74faae 100644 --- a/interfaces/innerkits/native_cpp/test/fuzztest/dscreen_source_proxy/proxyinitsource_fuzzer/initsource_fuzzer.cpp +++ b/interfaces/innerkits/native_cpp/test/fuzztest/dscreen_source_proxy/proxyinitsource_fuzzer/initsource_fuzzer.cpp @@ -34,7 +34,7 @@ void InitSourceFuzzTest(const uint8_t* data, size_t size) std::string params(reinterpret_cast(data), size); - sptr callback = new DScreenSourceCallback(); + sptr callback(new DScreenSourceCallback()); if (callback == nullptr) { return; } diff --git a/interfaces/innerkits/native_cpp/test/unittest/screensinktest/src/dscreen_sink_handler_test.cpp b/interfaces/innerkits/native_cpp/test/unittest/screensinktest/src/dscreen_sink_handler_test.cpp index decfb5ab27594805a0e880518eba1329e4b54d3b..7bae4e8277d922a8b2cb995e0bbdb8dd4de3d49e 100644 --- a/interfaces/innerkits/native_cpp/test/unittest/screensinktest/src/dscreen_sink_handler_test.cpp +++ b/interfaces/innerkits/native_cpp/test/unittest/screensinktest/src/dscreen_sink_handler_test.cpp @@ -78,7 +78,7 @@ HWTEST_F(DScreenSinkHandlerTest, LocalHardware_001, TestSize.Level1) const std::string param = "DScreenSinkHandlerTest"; int32_t ret = DScreenSinkHandler::GetInstance().InitSink(param); EXPECT_EQ(DH_SUCCESS, ret); - sptr loadCallback = new DScreenSinkLoadCallback(param); + sptr loadCallback(new DScreenSinkLoadCallback(param)); loadCallback->OnLoadSystemAbilitySuccess(DISTRIBUTED_HARDWARE_SCREEN_SINK_SA_ID, nullptr); loadCallback->OnLoadSystemAbilitySuccess(DISTRIBUTED_HARDWARE_SCREEN_SOURCE_SA_ID, nullptr); ret = DScreenSinkHandler::GetInstance().SubscribeLocalHardware(dhId, param); diff --git a/interfaces/innerkits/native_cpp/test/unittest/screensourcetest/src/dscreen_source_handler_test.cpp b/interfaces/innerkits/native_cpp/test/unittest/screensourcetest/src/dscreen_source_handler_test.cpp index 1c134b85fa591365a44da247c61325012e202a25..98d62ce9627b0094bfb191f4f109cabae42e62a4 100644 --- a/interfaces/innerkits/native_cpp/test/unittest/screensourcetest/src/dscreen_source_handler_test.cpp +++ b/interfaces/innerkits/native_cpp/test/unittest/screensourcetest/src/dscreen_source_handler_test.cpp @@ -84,7 +84,7 @@ HWTEST_F(DScreenSourceHandlerTest, RegisterDistributedHardware_001, TestSize.Lev param.version = "2.0"; param.attrs = "attrs"; std::string callbackParam = "callbackParam"; - sptr loadCallback = new DScreenSourceLoadCallback(callbackParam); + sptr loadCallback(new DScreenSourceLoadCallback(callbackParam)); loadCallback->OnLoadSystemAbilitySuccess(DISTRIBUTED_HARDWARE_SCREEN_SOURCE_SA_ID, nullptr); std::shared_ptr callback = std::make_shared(); int32_t ret = DScreenSourceHandler::GetInstance().RegisterDistributedHardware(devId, dhId, param, callback); @@ -105,7 +105,7 @@ HWTEST_F(DScreenSourceHandlerTest, RegisterDistributedHardware_002, TestSize.Lev param.version = "1"; param.attrs = "attrs"; std::string callbackParam = "callbackParam"; - sptr loadCallback = new DScreenSourceLoadCallback(callbackParam); + sptr loadCallback(new DScreenSourceLoadCallback(callbackParam)); loadCallback->OnLoadSystemAbilitySuccess(DISTRIBUTED_HARDWARE_SCREEN_SINK_SA_ID, nullptr); std::shared_ptr callback = std::make_shared(); DScreenSourceHandler::GetInstance().dScreenSourceProxy_ = nullptr; diff --git a/services/screenclient/src/screen_client_window_adapter.cpp b/services/screenclient/src/screen_client_window_adapter.cpp index 9759abec8b862408b5841ce717403d457194cf66..623c98f2a83d3c835e7acbfc567ddf8191d8cc5a 100644 --- a/services/screenclient/src/screen_client_window_adapter.cpp +++ b/services/screenclient/src/screen_client_window_adapter.cpp @@ -34,7 +34,7 @@ sptr ScreenClientWindowAdapter::CreateWindow(std::shared_ptr option = new Rosen::WindowOption(); + sptr option(new Rosen::WindowOption()); if (option == nullptr) { DHLOGE("option is nullptr."); return nullptr; diff --git a/services/screenservice/test/unittest/sinkservice/dscreenservice/src/dscreen_sink_stub_test.cpp b/services/screenservice/test/unittest/sinkservice/dscreenservice/src/dscreen_sink_stub_test.cpp index cff3b17a71d686f242079e6bb7661e6db1673d19..baab2b08e7febe9d2513e87d69e6fafbc7eb1e16 100644 --- a/services/screenservice/test/unittest/sinkservice/dscreenservice/src/dscreen_sink_stub_test.cpp +++ b/services/screenservice/test/unittest/sinkservice/dscreenservice/src/dscreen_sink_stub_test.cpp @@ -79,7 +79,7 @@ void DScreenSinkStubTest::TestDScreenSinkStub::DScreenNotify(const std::string & */ HWTEST_F(DScreenSinkStubTest, InitSink_001, TestSize.Level1) { - sptr sinkStubPtr = new TestDScreenSinkStub(); + sptr sinkStubPtr(new TestDScreenSinkStub()); DScreenSinkProxy sinkProxy(sinkStubPtr); int32_t ret = sinkProxy.InitSink("params000"); @@ -97,7 +97,7 @@ HWTEST_F(DScreenSinkStubTest, InitSink_001, TestSize.Level1) */ HWTEST_F(DScreenSinkStubTest, ReleaseSink_001, TestSize.Level1) { - sptr sinkStubPtr = new TestDScreenSinkStub(); + sptr sinkStubPtr(new TestDScreenSinkStub()); DScreenSinkProxy sinkProxy(sinkStubPtr); int32_t ret = sinkProxy.ReleaseSink(); @@ -112,7 +112,7 @@ HWTEST_F(DScreenSinkStubTest, ReleaseSink_001, TestSize.Level1) */ HWTEST_F(DScreenSinkStubTest, SubscribeLocalHardware_001, TestSize.Level1) { - sptr sinkStubPtr = new TestDScreenSinkStub(); + sptr sinkStubPtr(new TestDScreenSinkStub()); DScreenSinkProxy sinkProxy(sinkStubPtr); int32_t ret = sinkProxy.SubscribeLocalHardware("dhId000", "param000"); @@ -139,7 +139,7 @@ HWTEST_F(DScreenSinkStubTest, SubscribeLocalHardware_001, TestSize.Level1) */ HWTEST_F(DScreenSinkStubTest, UnsubscribeLocalHardware_001, TestSize.Level1) { - sptr sinkStubPtr = new TestDScreenSinkStub(); + sptr sinkStubPtr(new TestDScreenSinkStub()); DScreenSinkProxy sinkProxy(sinkStubPtr); int32_t ret = sinkProxy.UnsubscribeLocalHardware("dhId000"); @@ -163,7 +163,7 @@ HWTEST_F(DScreenSinkStubTest, UnsubscribeLocalHardware_001, TestSize.Level1) */ HWTEST_F(DScreenSinkStubTest, DScreenNotify_001, TestSize.Level1) { - sptr sinkStubPtr = new TestDScreenSinkStub(); + sptr sinkStubPtr(new TestDScreenSinkStub()); DScreenSinkProxy sinkProxy(sinkStubPtr); sinkProxy.DScreenNotify("", 0, "eventContent000"); diff --git a/services/screenservice/test/unittest/sourceservice/dscreenmgr/2.0/src/dscreen_manager_test.cpp b/services/screenservice/test/unittest/sourceservice/dscreenmgr/2.0/src/dscreen_manager_test.cpp index 780f30bc3fb27fd4df75ffbead6953f15a89ccd0..95f5924d5c66b358dc761e81aaa5bee032c58c5d 100644 --- a/services/screenservice/test/unittest/sourceservice/dscreenmgr/2.0/src/dscreen_manager_test.cpp +++ b/services/screenservice/test/unittest/sourceservice/dscreenmgr/2.0/src/dscreen_manager_test.cpp @@ -37,7 +37,7 @@ void DScreenManagerTestV2::TearDown() {} */ HWTEST_F(DScreenManagerTestV2, Initialize_001, TestSize.Level1) { - sptr dScreenGroupListener_ = new (std::nothrow) DScreenGroupListener(); + sptr dScreenGroupListener_(new (std::nothrow) DScreenGroupListener()); int32_t ret = DScreenManager::GetInstance().Initialize(); EXPECT_EQ(DH_SUCCESS, ret); } diff --git a/services/screenservice/test/unittest/sourceservice/dscreenservice/src/callback/dscreen_source_callback_stub_test.cpp b/services/screenservice/test/unittest/sourceservice/dscreenservice/src/callback/dscreen_source_callback_stub_test.cpp index e5c3c1dffffd78dd361f8ddba61451347c288f82..49231574c905d2dba90e71d6a69b43c3a2d9feb0 100644 --- a/services/screenservice/test/unittest/sourceservice/dscreenservice/src/callback/dscreen_source_callback_stub_test.cpp +++ b/services/screenservice/test/unittest/sourceservice/dscreenservice/src/callback/dscreen_source_callback_stub_test.cpp @@ -68,7 +68,7 @@ int32_t DScreenSourceCallbackStubTest::TestDScreenSourceCallbackStub::OnNotifyUn */ HWTEST_F(DScreenSourceCallbackStubTest, OnNotifyRegResult_001, TestSize.Level1) { - sptr sourceCbkStubPtr = new TestDScreenSourceCallbackStub(); + sptr sourceCbkStubPtr(new TestDScreenSourceCallbackStub()); DScreenSourceCallbackProxy sourceCbkProxy(sourceCbkStubPtr); int32_t ret = sourceCbkProxy.OnNotifyRegResult("", "dhId000", "reqId000", 0, "data000"); @@ -107,7 +107,7 @@ HWTEST_F(DScreenSourceCallbackStubTest, OnNotifyRegResult_001, TestSize.Level1) */ HWTEST_F(DScreenSourceCallbackStubTest, OnNotifyUnregResult_001, TestSize.Level1) { - sptr sourceCbkStubPtr = new TestDScreenSourceCallbackStub(); + sptr sourceCbkStubPtr(new TestDScreenSourceCallbackStub()); DScreenSourceCallbackProxy sourceCbkProxy(sourceCbkStubPtr); int32_t ret = sourceCbkProxy.OnNotifyUnregResult("", "dhId000", "reqId000", 0, "data000"); diff --git a/services/screenservice/test/unittest/sourceservice/dscreenservice/src/dscreen_source_stub_test.cpp b/services/screenservice/test/unittest/sourceservice/dscreenservice/src/dscreen_source_stub_test.cpp index 8828cf0bd1cafa6ec84112b98e5f7b9e99470589..b763d132e179049779ef87dcfab5ec7c5bdc8b75 100644 --- a/services/screenservice/test/unittest/sourceservice/dscreenservice/src/dscreen_source_stub_test.cpp +++ b/services/screenservice/test/unittest/sourceservice/dscreenservice/src/dscreen_source_stub_test.cpp @@ -96,7 +96,7 @@ void DScreenSourceStubTest::TestDScreenSourceStub::DScreenNotify(const std::stri */ HWTEST_F(DScreenSourceStubTest, InitSource_001, TestSize.Level1) { - sptr sourceStubPtr = new TestDScreenSourceStub(); + sptr sourceStubPtr(new TestDScreenSourceStub()); DScreenSourceProxy sourceProxy(sourceStubPtr); sptr callback; @@ -119,7 +119,7 @@ HWTEST_F(DScreenSourceStubTest, InitSource_001, TestSize.Level1) */ HWTEST_F(DScreenSourceStubTest, ReleaseSource_001, TestSize.Level1) { - sptr sourceStubPtr = new TestDScreenSourceStub(); + sptr sourceStubPtr(new TestDScreenSourceStub()); DScreenSourceProxy sourceProxy(sourceStubPtr); int32_t ret = sourceProxy.ReleaseSource(); EXPECT_EQ(DH_SUCCESS, ret); @@ -133,7 +133,7 @@ HWTEST_F(DScreenSourceStubTest, ReleaseSource_001, TestSize.Level1) */ HWTEST_F(DScreenSourceStubTest, RegisterDistributedHardware_001, TestSize.Level1) { - sptr sourceStubPtr = new TestDScreenSourceStub(); + sptr sourceStubPtr(new TestDScreenSourceStub()); DScreenSourceProxy sourceProxy(sourceStubPtr); EnableParam param; @@ -180,7 +180,7 @@ HWTEST_F(DScreenSourceStubTest, RegisterDistributedHardware_001, TestSize.Level1 */ HWTEST_F(DScreenSourceStubTest, UnregisterDistributedHardware_001, TestSize.Level1) { - sptr sourceStubPtr = new TestDScreenSourceStub(); + sptr sourceStubPtr(new TestDScreenSourceStub()); DScreenSourceProxy sourceProxy(sourceStubPtr); int32_t ret = sourceProxy.UnregisterDistributedHardware("", "dhId000", "reqId000"); @@ -216,7 +216,7 @@ HWTEST_F(DScreenSourceStubTest, UnregisterDistributedHardware_001, TestSize.Leve */ HWTEST_F(DScreenSourceStubTest, ConfigDistributedHardware_001, TestSize.Level1) { - sptr sourceStubPtr = new TestDScreenSourceStub(); + sptr sourceStubPtr(new TestDScreenSourceStub()); DScreenSourceProxy sourceProxy(sourceStubPtr); int32_t ret = sourceProxy.ConfigDistributedHardware("", "dhId000", "key000", "value000"); @@ -252,7 +252,7 @@ HWTEST_F(DScreenSourceStubTest, ConfigDistributedHardware_001, TestSize.Level1) */ HWTEST_F(DScreenSourceStubTest, DScreenNotify_001, TestSize.Level1) { - sptr sourceStubPtr = new TestDScreenSourceStub(); + sptr sourceStubPtr(new TestDScreenSourceStub()); DScreenSourceProxy sourceProxy(sourceStubPtr); sourceProxy.DScreenNotify("", 0, "eventContent000");