diff --git a/services/native/file_access_service/src/file_access_service.cpp b/services/native/file_access_service/src/file_access_service.cpp index 46d204ac7f833644cc8afbb6152777e24e7e57d8..063c8627199d311390c96fdcdeb85f2894028d66 100644 --- a/services/native/file_access_service/src/file_access_service.cpp +++ b/services/native/file_access_service/src/file_access_service.cpp @@ -54,7 +54,7 @@ sptr FileAccessService::GetInstance() lock_guard lock(mutex_); if (instance_ == nullptr) { - instance_ = new FileAccessService(); + instance_ = sptr(new FileAccessService()); if (instance_ == nullptr) { HILOG_ERROR("GetInstance nullptr"); return instance_; @@ -136,10 +136,10 @@ void FileAccessService::Init() { InitTimer(); if (extensionDeathRecipient_ == nullptr) { - extensionDeathRecipient_ = new ExtensionDeathRecipient(); + extensionDeathRecipient_ = sptr(new ExtensionDeathRecipient()); } if (observerDeathRecipient_ == nullptr) { - observerDeathRecipient_ = new ObserverDeathRecipient(); + observerDeathRecipient_ = sptr(new ObserverDeathRecipient()); } } diff --git a/test/fuzztest/externalfileaccess_fuzzer/external_file_access_fuzzer.cpp b/test/fuzztest/externalfileaccess_fuzzer/external_file_access_fuzzer.cpp index 8dc8a282ceecd4e16cbb0f2bb1589115fbaa33ef..46a7774f829e751d97607632c935fbccede8fa1e 100644 --- a/test/fuzztest/externalfileaccess_fuzzer/external_file_access_fuzzer.cpp +++ b/test/fuzztest/externalfileaccess_fuzzer/external_file_access_fuzzer.cpp @@ -41,12 +41,13 @@ const int UID_DEFAULT = 0; void SetNativeToken() { uint64_t tokenId; - const char **perms = new const char *[2]; + const char **perms = new const char *[3]; perms[0] = "ohos.permission.FILE_ACCESS_MANAGER"; perms[1] = "ohos.permission.GET_BUNDLE_INFO_PRIVILEGED"; + perms[2] = "ohos.permission.CONNECT_FILE_ACCESS_EXTENSION"; NativeTokenInfoParams infoInstance = { .dcapsNum = 0, - .permsNum = 2, + .permsNum = 3, .aclsNum = 0, .dcaps = nullptr, .perms = perms, diff --git a/test/fuzztest/externalfileaccessaccess_fuzzer/externalfileaccessaccess_fuzzer.cpp b/test/fuzztest/externalfileaccessaccess_fuzzer/externalfileaccessaccess_fuzzer.cpp index 484869411da3b16b51cc756ebb029cfde883f6be..18ce5d32ad9590a1532e7c0ffed887f6803e39ff 100644 --- a/test/fuzztest/externalfileaccessaccess_fuzzer/externalfileaccessaccess_fuzzer.cpp +++ b/test/fuzztest/externalfileaccessaccess_fuzzer/externalfileaccessaccess_fuzzer.cpp @@ -48,12 +48,13 @@ enum { void SetNativeToken() { uint64_t tokenId; - const char **perms = new const char *[2]; + const char **perms = new const char *[3]; perms[0] = "ohos.permission.FILE_ACCESS_MANAGER"; perms[1] = "ohos.permission.GET_BUNDLE_INFO_PRIVILEGED"; + perms[2] = "ohos.permission.CONNECT_FILE_ACCESS_EXTENSION"; NativeTokenInfoParams infoInstance = { .dcapsNum = 0, - .permsNum = 2, + .permsNum = 3, .aclsNum = 0, .dcaps = nullptr, .perms = perms, diff --git a/test/fuzztest/externalfileaccesscreatefile_fuzzer/externalfileaccesscreatefile_fuzzer.cpp b/test/fuzztest/externalfileaccesscreatefile_fuzzer/externalfileaccesscreatefile_fuzzer.cpp index 4c7d4c0bb444654e0e77bb8d2cbd48e243277e2f..c448bbf50ffa330c6991002b48c43bdc29fc5d52 100644 --- a/test/fuzztest/externalfileaccesscreatefile_fuzzer/externalfileaccesscreatefile_fuzzer.cpp +++ b/test/fuzztest/externalfileaccesscreatefile_fuzzer/externalfileaccesscreatefile_fuzzer.cpp @@ -48,12 +48,13 @@ enum { void SetNativeToken() { uint64_t tokenId; - const char **perms = new const char *[2]; + const char **perms = new const char *[3]; perms[0] = "ohos.permission.FILE_ACCESS_MANAGER"; perms[1] = "ohos.permission.GET_BUNDLE_INFO_PRIVILEGED"; + perms[2] = "ohos.permission.CONNECT_FILE_ACCESS_EXTENSION"; NativeTokenInfoParams infoInstance = { .dcapsNum = 0, - .permsNum = 2, + .permsNum = 3, .aclsNum = 0, .dcaps = nullptr, .perms = perms, diff --git a/test/fuzztest/externalfileaccessdelete_fuzzer/externalfileaccessdelete_fuzzer.cpp b/test/fuzztest/externalfileaccessdelete_fuzzer/externalfileaccessdelete_fuzzer.cpp index 568e3fca773124b0774e7ae8d6aa8d57b3496377..cbcdfa466c0d9ad6a234c04b95f8fc610785a65c 100644 --- a/test/fuzztest/externalfileaccessdelete_fuzzer/externalfileaccessdelete_fuzzer.cpp +++ b/test/fuzztest/externalfileaccessdelete_fuzzer/externalfileaccessdelete_fuzzer.cpp @@ -48,12 +48,13 @@ enum { void SetNativeToken() { uint64_t tokenId; - const char **perms = new const char *[2]; + const char **perms = new const char *[3]; perms[0] = "ohos.permission.FILE_ACCESS_MANAGER"; perms[1] = "ohos.permission.GET_BUNDLE_INFO_PRIVILEGED"; + perms[2] = "ohos.permission.CONNECT_FILE_ACCESS_EXTENSION"; NativeTokenInfoParams infoInstance = { .dcapsNum = 0, - .permsNum = 2, + .permsNum = 3, .aclsNum = 0, .dcaps = nullptr, .perms = perms, diff --git a/test/fuzztest/externalfileaccessgetfileinfofromrelativepath_fuzzer/externalfileaccessgetfileinfofromrelativepath_fuzzer.cpp b/test/fuzztest/externalfileaccessgetfileinfofromrelativepath_fuzzer/externalfileaccessgetfileinfofromrelativepath_fuzzer.cpp index 92a53e858924df0a1e5edf9caf1fdf9143373c4a..9c485975f69bc87b06ecada92180cabb21274c81 100644 --- a/test/fuzztest/externalfileaccessgetfileinfofromrelativepath_fuzzer/externalfileaccessgetfileinfofromrelativepath_fuzzer.cpp +++ b/test/fuzztest/externalfileaccessgetfileinfofromrelativepath_fuzzer/externalfileaccessgetfileinfofromrelativepath_fuzzer.cpp @@ -48,12 +48,13 @@ enum { void SetNativeToken() { uint64_t tokenId; - const char **perms = new const char *[2]; + const char **perms = new const char *[3]; perms[0] = "ohos.permission.FILE_ACCESS_MANAGER"; perms[1] = "ohos.permission.GET_BUNDLE_INFO_PRIVILEGED"; + perms[2] = "ohos.permission.CONNECT_FILE_ACCESS_EXTENSION"; NativeTokenInfoParams infoInstance = { .dcapsNum = 0, - .permsNum = 2, + .permsNum = 3, .aclsNum = 0, .dcaps = nullptr, .perms = perms, diff --git a/test/fuzztest/externalfileaccessgetfileinfofromuri_fuzzer/externalfileaccessgetfileinfofromuri_fuzzer.cpp b/test/fuzztest/externalfileaccessgetfileinfofromuri_fuzzer/externalfileaccessgetfileinfofromuri_fuzzer.cpp index b1c345c20759ab4d1f3a6c4451fa3102bbb2b61e..3ac2b456f8f48b66a0783db5e06ca9ee2048ab88 100644 --- a/test/fuzztest/externalfileaccessgetfileinfofromuri_fuzzer/externalfileaccessgetfileinfofromuri_fuzzer.cpp +++ b/test/fuzztest/externalfileaccessgetfileinfofromuri_fuzzer/externalfileaccessgetfileinfofromuri_fuzzer.cpp @@ -48,12 +48,13 @@ enum { void SetNativeToken() { uint64_t tokenId; - const char **perms = new const char *[2]; + const char **perms = new const char *[3]; perms[0] = "ohos.permission.FILE_ACCESS_MANAGER"; perms[1] = "ohos.permission.GET_BUNDLE_INFO_PRIVILEGED"; + perms[2] = "ohos.permission.CONNECT_FILE_ACCESS_EXTENSION"; NativeTokenInfoParams infoInstance = { .dcapsNum = 0, - .permsNum = 2, + .permsNum = 3, .aclsNum = 0, .dcaps = nullptr, .perms = perms, diff --git a/test/fuzztest/externalfileaccessgetroots_fuzzer/externalfileaccessgetroots_fuzzer.cpp b/test/fuzztest/externalfileaccessgetroots_fuzzer/externalfileaccessgetroots_fuzzer.cpp index 324caba190904d2a4370c61a15472c40a32ad1b1..8dc95df8d13fe15043d09161d40b5cdef21b2f5e 100644 --- a/test/fuzztest/externalfileaccessgetroots_fuzzer/externalfileaccessgetroots_fuzzer.cpp +++ b/test/fuzztest/externalfileaccessgetroots_fuzzer/externalfileaccessgetroots_fuzzer.cpp @@ -48,12 +48,13 @@ enum { void SetNativeToken() { uint64_t tokenId; - const char **perms = new const char *[2]; + const char **perms = new const char *[3]; perms[0] = "ohos.permission.FILE_ACCESS_MANAGER"; perms[1] = "ohos.permission.GET_BUNDLE_INFO_PRIVILEGED"; + perms[2] = "ohos.permission.CONNECT_FILE_ACCESS_EXTENSION"; NativeTokenInfoParams infoInstance = { .dcapsNum = 0, - .permsNum = 2, + .permsNum = 3, .aclsNum = 0, .dcaps = nullptr, .perms = perms, diff --git a/test/fuzztest/externalfileaccesslistfile_fuzzer/externalfileaccesslistfile_fuzzer.cpp b/test/fuzztest/externalfileaccesslistfile_fuzzer/externalfileaccesslistfile_fuzzer.cpp index bb48e0d2c26c621029f307dea80315daee73636d..6ed737e0dc5aa1ce60a31f376a8e86e55d47b2d3 100644 --- a/test/fuzztest/externalfileaccesslistfile_fuzzer/externalfileaccesslistfile_fuzzer.cpp +++ b/test/fuzztest/externalfileaccesslistfile_fuzzer/externalfileaccesslistfile_fuzzer.cpp @@ -48,12 +48,13 @@ enum { void SetNativeToken() { uint64_t tokenId; - const char **perms = new const char *[2]; + const char **perms = new const char *[3]; perms[0] = "ohos.permission.FILE_ACCESS_MANAGER"; perms[1] = "ohos.permission.GET_BUNDLE_INFO_PRIVILEGED"; + perms[2] = "ohos.permission.CONNECT_FILE_ACCESS_EXTENSION"; NativeTokenInfoParams infoInstance = { .dcapsNum = 0, - .permsNum = 2, + .permsNum = 3, .aclsNum = 0, .dcaps = nullptr, .perms = perms, diff --git a/test/fuzztest/externalfileaccessmkdir_fuzzer/externalfileaccessmkdir_fuzzer.cpp b/test/fuzztest/externalfileaccessmkdir_fuzzer/externalfileaccessmkdir_fuzzer.cpp index fa7880c0a66ea25db201d7bf036d1b5dec691415..d3cfc9f0ef4bcadfdc6d9be4426e5b345a0988a0 100644 --- a/test/fuzztest/externalfileaccessmkdir_fuzzer/externalfileaccessmkdir_fuzzer.cpp +++ b/test/fuzztest/externalfileaccessmkdir_fuzzer/externalfileaccessmkdir_fuzzer.cpp @@ -48,12 +48,13 @@ enum { void SetNativeToken() { uint64_t tokenId; - const char **perms = new const char *[2]; + const char **perms = new const char *[3]; perms[0] = "ohos.permission.FILE_ACCESS_MANAGER"; perms[1] = "ohos.permission.GET_BUNDLE_INFO_PRIVILEGED"; + perms[2] = "ohos.permission.CONNECT_FILE_ACCESS_EXTENSION"; NativeTokenInfoParams infoInstance = { .dcapsNum = 0, - .permsNum = 2, + .permsNum = 3, .aclsNum = 0, .dcaps = nullptr, .perms = perms, diff --git a/test/fuzztest/externalfileaccessmove_fuzzer/externalfileaccessmove_fuzzer.cpp b/test/fuzztest/externalfileaccessmove_fuzzer/externalfileaccessmove_fuzzer.cpp index 2d1eea90b0399baf8f4d87e86a642f5c2c51c598..260a8f040e36c22e457707216cd57e6d33d399fe 100644 --- a/test/fuzztest/externalfileaccessmove_fuzzer/externalfileaccessmove_fuzzer.cpp +++ b/test/fuzztest/externalfileaccessmove_fuzzer/externalfileaccessmove_fuzzer.cpp @@ -48,12 +48,13 @@ enum { void SetNativeToken() { uint64_t tokenId; - const char **perms = new const char *[2]; + const char **perms = new const char *[3]; perms[0] = "ohos.permission.FILE_ACCESS_MANAGER"; perms[1] = "ohos.permission.GET_BUNDLE_INFO_PRIVILEGED"; + perms[2] = "ohos.permission.CONNECT_FILE_ACCESS_EXTENSION"; NativeTokenInfoParams infoInstance = { .dcapsNum = 0, - .permsNum = 2, + .permsNum = 3, .aclsNum = 0, .dcaps = nullptr, .perms = perms, diff --git a/test/fuzztest/externalfileaccessopenfile_fuzzer/externalfileaccessopenfile_fuzzer.cpp b/test/fuzztest/externalfileaccessopenfile_fuzzer/externalfileaccessopenfile_fuzzer.cpp index 90af01bc00785689bae34a9d4726148f6ceb29b5..ccef60b28dd7d31699ff237398d4fa0e73349249 100644 --- a/test/fuzztest/externalfileaccessopenfile_fuzzer/externalfileaccessopenfile_fuzzer.cpp +++ b/test/fuzztest/externalfileaccessopenfile_fuzzer/externalfileaccessopenfile_fuzzer.cpp @@ -48,12 +48,13 @@ enum { void SetNativeToken() { uint64_t tokenId; - const char **perms = new const char *[2]; + const char **perms = new const char *[3]; perms[0] = "ohos.permission.FILE_ACCESS_MANAGER"; perms[1] = "ohos.permission.GET_BUNDLE_INFO_PRIVILEGED"; + perms[2] = "ohos.permission.CONNECT_FILE_ACCESS_EXTENSION"; NativeTokenInfoParams infoInstance = { .dcapsNum = 0, - .permsNum = 2, + .permsNum = 3, .aclsNum = 0, .dcaps = nullptr, .perms = perms, diff --git a/test/fuzztest/externalfileaccessrename_fuzzer/externalfileaccessrename_fuzzer.cpp b/test/fuzztest/externalfileaccessrename_fuzzer/externalfileaccessrename_fuzzer.cpp index 9caa7d5360d35f1f424887663bdf0a6c0f5fc565..5c1a72d1a5fa04950ecbe5edea07bcfe25849ab2 100644 --- a/test/fuzztest/externalfileaccessrename_fuzzer/externalfileaccessrename_fuzzer.cpp +++ b/test/fuzztest/externalfileaccessrename_fuzzer/externalfileaccessrename_fuzzer.cpp @@ -48,12 +48,13 @@ enum { void SetNativeToken() { uint64_t tokenId; - const char **perms = new const char *[2]; + const char **perms = new const char *[3]; perms[0] = "ohos.permission.FILE_ACCESS_MANAGER"; perms[1] = "ohos.permission.GET_BUNDLE_INFO_PRIVILEGED"; + perms[2] = "ohos.permission.CONNECT_FILE_ACCESS_EXTENSION"; NativeTokenInfoParams infoInstance = { .dcapsNum = 0, - .permsNum = 2, + .permsNum = 3, .aclsNum = 0, .dcaps = nullptr, .perms = perms, diff --git a/test/fuzztest/externalfileaccessscanfile_fuzzer/externalfileaccessscanfile_fuzzer.cpp b/test/fuzztest/externalfileaccessscanfile_fuzzer/externalfileaccessscanfile_fuzzer.cpp index 991a5f844fc1ee8dc6f9f7800c214772cf52fb6c..d85821a5eb8362928cb1601bf66ba1b31f19b2a1 100644 --- a/test/fuzztest/externalfileaccessscanfile_fuzzer/externalfileaccessscanfile_fuzzer.cpp +++ b/test/fuzztest/externalfileaccessscanfile_fuzzer/externalfileaccessscanfile_fuzzer.cpp @@ -48,12 +48,13 @@ enum { void SetNativeToken() { uint64_t tokenId; - const char **perms = new const char *[2]; + const char **perms = new const char *[3]; perms[0] = "ohos.permission.FILE_ACCESS_MANAGER"; perms[1] = "ohos.permission.GET_BUNDLE_INFO_PRIVILEGED"; + perms[2] = "ohos.permission.CONNECT_FILE_ACCESS_EXTENSION"; NativeTokenInfoParams infoInstance = { .dcapsNum = 0, - .permsNum = 2, + .permsNum = 3, .aclsNum = 0, .dcaps = nullptr, .perms = perms, diff --git a/test/fuzztest/fileaccessserviceproxy_fuzzer/fileaccessserviceproxy_fuzzer.cpp b/test/fuzztest/fileaccessserviceproxy_fuzzer/fileaccessserviceproxy_fuzzer.cpp index a2f8c00d8a2abc6921f66e562cd6221290e9d8d6..039d075bce5b4a3e60ed51a3ef0c7a6dadc8c0e4 100644 --- a/test/fuzztest/fileaccessserviceproxy_fuzzer/fileaccessserviceproxy_fuzzer.cpp +++ b/test/fuzztest/fileaccessserviceproxy_fuzzer/fileaccessserviceproxy_fuzzer.cpp @@ -28,7 +28,7 @@ using namespace FileAccessFwk; SystemAbilityManagerClient& SystemAbilityManagerClient::GetInstance() { - static auto instance = new SystemAbilityManagerClient(); + static auto instance = std::make_shared(); return *instance; } diff --git a/test/fuzztest/medialibraryfileaccess_fuzzer/medialibrary_file_access_fuzzer.cpp b/test/fuzztest/medialibraryfileaccess_fuzzer/medialibrary_file_access_fuzzer.cpp index 52ce4e2d1beb53ed7bea03919a787f14bbb0865c..f8d8e635010ee6cd07a39c87156a79032582c8e0 100644 --- a/test/fuzztest/medialibraryfileaccess_fuzzer/medialibrary_file_access_fuzzer.cpp +++ b/test/fuzztest/medialibraryfileaccess_fuzzer/medialibrary_file_access_fuzzer.cpp @@ -41,12 +41,13 @@ const int UID_DEFAULT = 0; void SetNativeToken() { uint64_t tokenId; - const char **perms = new const char *[2]; + const char **perms = new const char *[3]; perms[0] = "ohos.permission.FILE_ACCESS_MANAGER"; perms[1] = "ohos.permission.GET_BUNDLE_INFO_PRIVILEGED"; + perms[2] = "ohos.permission.CONNECT_FILE_ACCESS_EXTENSION"; NativeTokenInfoParams infoInstance = { .dcapsNum = 0, - .permsNum = 2, + .permsNum = 3, .aclsNum = 0, .dcaps = nullptr, .perms = perms, diff --git a/test/unittest/abnormal_file_access_test.cpp b/test/unittest/abnormal_file_access_test.cpp index 15dedcb6dc6384e137c9df6066e9b8bbaa7c8e35..fff3fd460555c2ac1ea4bfa6d11a3f7df127f18a 100755 --- a/test/unittest/abnormal_file_access_test.cpp +++ b/test/unittest/abnormal_file_access_test.cpp @@ -40,12 +40,13 @@ shared_ptr g_context = nullptr; static void SetNativeToken(bool isSystemApp) { uint64_t tokenId; - const char **perms = new const char *[2]; + const char **perms = new const char *[3]; perms[0] = "ohos.permission.FILE_ACCESS_MANAGER"; perms[1] = "ohos.permission.GET_BUNDLE_INFO_PRIVILEGED"; + perms[2] = "ohos.permission.CONNECT_FILE_ACCESS_EXTENSION"; NativeTokenInfoParams infoInstance = { .dcapsNum = 0, - .permsNum = 2, + .permsNum = 3, .aclsNum = 0, .dcaps = nullptr, .perms = perms, diff --git a/test/unittest/external_file_access_test.cpp b/test/unittest/external_file_access_test.cpp index 8695d627131a03235c989759a6296b7549c42b61..6445688343dbc30e0296e7d287936631e1b740e4 100644 --- a/test/unittest/external_file_access_test.cpp +++ b/test/unittest/external_file_access_test.cpp @@ -36,20 +36,18 @@ static shared_ptr g_context = nullptr; void SetNativeToken() { uint64_t tokenId; - const char **perms = new const char *[2]; - if (perms == nullptr) { - return; - } + const char **perms = new const char *[3]; perms[0] = "ohos.permission.FILE_ACCESS_MANAGER"; perms[1] = "ohos.permission.GET_BUNDLE_INFO_PRIVILEGED"; + perms[2] = "ohos.permission.CONNECT_FILE_ACCESS_EXTENSION"; NativeTokenInfoParams infoInstance = { - .aplStr = "system_core", - .acls = nullptr, .dcapsNum = 0, - .permsNum = 2, + .permsNum = 3, .aclsNum = 0, .dcaps = nullptr, .perms = perms, + .acls = nullptr, + .aplStr = "system_core", }; infoInstance.processName = "SetUpTestCase"; diff --git a/test/unittest/external_notify_test.cpp b/test/unittest/external_notify_test.cpp index f37092a2695e85bc96e2af95d607853563d2299f..98a66b942119d519d8029a83cec21ab5bfd7e3cf 100644 --- a/test/unittest/external_notify_test.cpp +++ b/test/unittest/external_notify_test.cpp @@ -56,12 +56,13 @@ shared_ptr g_context = nullptr; void SetNativeToken() { uint64_t tokenId; - const char **perms = new const char *[2]; + const char **perms = new const char *[3]; perms[0] = "ohos.permission.FILE_ACCESS_MANAGER"; perms[1] = "ohos.permission.GET_BUNDLE_INFO_PRIVILEGED"; + perms[2] = "ohos.permission.CONNECT_FILE_ACCESS_EXTENSION"; NativeTokenInfoParams infoInstance = { .dcapsNum = 0, - .permsNum = 2, + .permsNum = 3, .aclsNum = 0, .dcaps = nullptr, .perms = perms, @@ -263,10 +264,10 @@ HWTEST_F(FileExtensionNotifyTest, external_file_access_notify_0000, testing::ext EXPECT_NE(g_fah, nullptr); int result = g_fah->GetRoots(info); EXPECT_EQ(result, OHOS::FileAccessFwk::ERR_OK); - sptr myObserver1 = new (std::nothrow) MyObserver(); - sptr myObserver2 = new (std::nothrow) MyObserver(); - sptr myObserver3 = new (std::nothrow) MyObserver(); - sptr myObserver4 = new (std::nothrow) MyObserver(); + sptr myObserver1 = sptr(new (std::nothrow) MyObserver()); + sptr myObserver2 = sptr(new (std::nothrow) MyObserver()); + sptr myObserver3 = sptr(new (std::nothrow) MyObserver()); + sptr myObserver4 = sptr(new (std::nothrow) MyObserver()); Uri parentUri(info[1].uri); auto [newDirUriTest1, newDirUriTest2, newFileUri1, newFileUri2] = ReadyRegisterNotify00(parentUri, myObserver1, myObserver2, myObserver3, myObserver4); @@ -335,9 +336,9 @@ HWTEST_F(FileExtensionNotifyTest, external_file_access_notify_0001, testing::ext int result = g_fah->GetRoots(info); EXPECT_EQ(result, OHOS::FileAccessFwk::ERR_OK); - sptr myObserver1 = new (std::nothrow) MyObserver(); - sptr myObserver2 = new (std::nothrow) MyObserver(); - sptr myObserver3 = new (std::nothrow) MyObserver(); + sptr myObserver1 = sptr(new (std::nothrow) MyObserver()); + sptr myObserver2 = sptr(new (std::nothrow) MyObserver()); + sptr myObserver3 = sptr(new (std::nothrow) MyObserver()); Uri parentUri(info[1].uri); auto [newFileUri1, newFileUri2, newFileUri3] = ReadyRegisterNotify01(parentUri, myObserver1, myObserver2, myObserver3); @@ -506,8 +507,8 @@ HWTEST_F(FileExtensionNotifyTest, external_file_access_notify_0003, testing::ext int result = g_fah->GetRoots(info); EXPECT_EQ(result, OHOS::FileAccessFwk::ERR_OK); bool notifyForDescendants = true; - sptr myObserver1 = new (std::nothrow) MyObserver(); - sptr myObserver2 = new (std::nothrow) MyObserver(); + sptr myObserver1 = sptr(new (std::nothrow) MyObserver()); + sptr myObserver2 = sptr(new (std::nothrow) MyObserver()); Uri parentUri(info[1].uri); Uri newDirUriTest1(""); result = g_fah->Mkdir(parentUri, "uri_dir1", newDirUriTest1); @@ -584,7 +585,7 @@ HWTEST_F(FileExtensionNotifyTest, external_file_access_notify_0005, testing::ext EXPECT_NE(g_fah, nullptr); int result = g_fah->GetRoots(info); EXPECT_EQ(result, OHOS::FileAccessFwk::ERR_OK); - sptr myObserver1 = new (std::nothrow) MyObserver(); + sptr myObserver1 = sptr(new (std::nothrow) MyObserver()); Uri parentUri(info[1].uri); auto [uri_dir, uri_dirSub1, uri_dirSub2, renameDirUri1] = ReadyRegisterNotify05(parentUri, myObserver1); if (g_notifyEvent != MOVED_TO) { @@ -637,7 +638,7 @@ HWTEST_F(FileExtensionNotifyTest, external_file_access_notify_0006, testing::ext int result = g_fah->GetRoots(info); EXPECT_EQ(result, OHOS::FileAccessFwk::ERR_OK); bool notifyForDescendants = true; - sptr myObserver1 = new (std::nothrow) MyObserver(); + sptr myObserver1 = sptr(new (std::nothrow) MyObserver()); Uri parentUri(info[1].uri); Uri newFileUri1(""); result = g_fah->CreateFile(parentUri, "uri_file", newFileUri1); @@ -712,7 +713,7 @@ HWTEST_F(FileExtensionNotifyTest, external_file_access_notify_0008, testing::ext EXPECT_NE(g_fah, nullptr); int result = g_fah->GetRoots(info); EXPECT_EQ(result, OHOS::FileAccessFwk::ERR_OK); - sptr myObserver1 = new (std::nothrow) MyObserver(); + sptr myObserver1 = sptr(new (std::nothrow) MyObserver()); Uri newFileUri1("*/&%"); bool notifyForDescendants = true; result = g_fah->RegisterNotify(newFileUri1, notifyForDescendants, myObserver1); @@ -743,7 +744,7 @@ HWTEST_F(FileExtensionNotifyTest, external_file_access_notify_0009, testing::ext int result = g_fah->GetRoots(info); EXPECT_EQ(result, OHOS::FileAccessFwk::ERR_OK); bool notifyForDescendants = true; - sptr myObserver1 = new (std::nothrow) MyObserver(); + sptr myObserver1 = sptr(new (std::nothrow) MyObserver()); Uri parentUri(info[1].uri); Uri newFileUri1(""); result = g_fah->CreateFile(parentUri, "测试文件", newFileUri1); @@ -782,8 +783,8 @@ HWTEST_F(FileExtensionNotifyTest, external_file_access_notify_0010, testing::ext int result = g_fah->GetRoots(info); EXPECT_EQ(result, OHOS::FileAccessFwk::ERR_OK); bool notifyForDescendants = true; - sptr myObserver1 = new (std::nothrow) MyObserver(); - sptr myObserver2 = new (std::nothrow) MyObserver(); + sptr myObserver1 = sptr(new (std::nothrow) MyObserver()); + sptr myObserver2 = sptr(new (std::nothrow) MyObserver()); Uri parentUri(info[1].uri); Uri newFileUri1(""); result = g_fah->CreateFile(parentUri, "uri_file1", newFileUri1); @@ -855,8 +856,8 @@ HWTEST_F(FileExtensionNotifyTest, external_file_access_notify_0011, testing::ext int result = g_fah->GetRoots(info); EXPECT_EQ(result, OHOS::FileAccessFwk::ERR_OK); bool notifyForDescendants2 = false; - sptr myObserver1 = new (std::nothrow) TestObserver(); - sptr myObserver2 = new (std::nothrow) MyObserver(); + sptr myObserver1 = sptr(new (std::nothrow) TestObserver()); + sptr myObserver2 = sptr(new (std::nothrow) MyObserver()); Uri parentUri(info[1].uri); Uri newFileDir1(""); auto [newFileUri1, renameFileUri1] = @@ -936,7 +937,7 @@ HWTEST_F(FileExtensionNotifyTest, external_file_access_notify_00012, testing::ex Uri parentUri(info[1].uri); Uri newFileUri1(""); GTEST_LOG_(INFO) << parentUri.ToString(); - sptr myObserver1 = new (std::nothrow) MyObserver(); + sptr myObserver1 = sptr(new (std::nothrow) MyObserver()); for (int i = 0; i < INIT_THREADS_NUMBER; i++) { std::thread execthread1(RegisterDirNotify, parentUri, "WatcherTest", myObserver1.GetRefPtr()); execthread1.join(); @@ -966,7 +967,7 @@ HWTEST_F(FileExtensionNotifyTest, external_file_access_notify_00013, testing::ex EXPECT_EQ(result, OHOS::FileAccessFwk::ERR_OK); Uri parentUri(info[1].uri); GTEST_LOG_(INFO) << parentUri.ToString(); - sptr myObserver1 = new (std::nothrow) MyObserver(); + sptr myObserver1 = sptr(new (std::nothrow) MyObserver()); std::thread execthread1(RegisterDirNotify, parentUri, "WatcherTest1", myObserver1.GetRefPtr()); execthread1.join(); std::thread execthread2(RegisterDirNotify, parentUri, "WatcherTest2", myObserver1.GetRefPtr()); @@ -998,9 +999,9 @@ HWTEST_F(FileExtensionNotifyTest, external_file_access_notify_00014, testing::ex EXPECT_EQ(result, OHOS::FileAccessFwk::ERR_OK); Uri parentUri(info[1].uri); GTEST_LOG_(INFO) << parentUri.ToString(); - sptr myObserver1 = new (std::nothrow) MyObserver(); - sptr myObserver2 = new (std::nothrow) MyObserver(); - sptr myObserver3 = new (std::nothrow) MyObserver(); + sptr myObserver1 = sptr(new (std::nothrow) MyObserver()); + sptr myObserver2 = sptr(new (std::nothrow) MyObserver()); + sptr myObserver3 = sptr(new (std::nothrow) MyObserver()); std::thread execthread1(RegisterDirNotify, parentUri, "WatcherTest", myObserver1.GetRefPtr()); execthread1.join(); @@ -1035,9 +1036,9 @@ HWTEST_F(FileExtensionNotifyTest, external_file_access_notify_00015, testing::ex EXPECT_EQ(result, OHOS::FileAccessFwk::ERR_OK); Uri parentUri(info[1].uri); GTEST_LOG_(INFO) << parentUri.ToString(); - sptr myObserver1 = new (std::nothrow) MyObserver(); - sptr myObserver2 = new (std::nothrow) MyObserver(); - sptr myObserver3 = new (std::nothrow) MyObserver(); + sptr myObserver1 = sptr(new (std::nothrow) MyObserver()); + sptr myObserver2 = sptr(new (std::nothrow) MyObserver()); + sptr myObserver3 = sptr(new (std::nothrow) MyObserver()); std::thread execthread1(RegisterDirNotify, parentUri, "WatcherTest1", myObserver1.GetRefPtr()); execthread1.join(); @@ -1069,9 +1070,9 @@ HWTEST_F(FileExtensionNotifyTest, external_file_access_notify_0016, testing::ext int result = g_fah->GetRoots(info); EXPECT_EQ(result, OHOS::FileAccessFwk::ERR_OK); bool notifyForDescendants = true; - sptr myObserver1 = new (std::nothrow) MyObserver(); - sptr myObserver2 = new (std::nothrow) MyObserver(); - sptr myObserver3 = new (std::nothrow) MyObserver(); + sptr myObserver1 = sptr(new (std::nothrow) MyObserver()); + sptr myObserver2 = sptr(new (std::nothrow) MyObserver()); + sptr myObserver3 = sptr(new (std::nothrow) MyObserver()); Uri parentUri(info[1].uri); Uri newFileDir1(""); result = g_fah->Mkdir(parentUri, "uri_dir0016", newFileDir1); @@ -1114,7 +1115,7 @@ HWTEST_F(FileExtensionNotifyTest, external_file_access_notify_0017, testing::ext int result = g_fah->GetRoots(info); EXPECT_EQ(result, OHOS::FileAccessFwk::ERR_OK); bool notifyForDescendants = true; - sptr myObserver1 = new (std::nothrow) MyObserver(); + sptr myObserver1 = sptr(new (std::nothrow) MyObserver()); Uri parentUri(info[1].uri); Uri newFileDir1(""); result = g_fah->Mkdir(parentUri, "uri_dir", newFileDir1); @@ -1158,7 +1159,7 @@ HWTEST_F(FileExtensionNotifyTest, external_file_access_notify_0018, testing::ext int result = g_fah->GetRoots(info); EXPECT_EQ(result, OHOS::FileAccessFwk::ERR_OK); bool notifyForDescendants = true; - sptr myObserver1 = new (std::nothrow) TestObserver(); + sptr myObserver1 = sptr(new (std::nothrow) TestObserver()); Uri parentUri(info[1].uri); Uri newFileDir1(""); result = g_fah->Mkdir(parentUri, "uri_dir", newFileDir1); @@ -1202,7 +1203,7 @@ HWTEST_F(FileExtensionNotifyTest, external_file_access_notify_0019, testing::ext int result = g_fah->GetRoots(info); EXPECT_EQ(result, OHOS::FileAccessFwk::ERR_OK); bool notifyForDescendants = true; - sptr myObserver1 = new (std::nothrow) TestObserver(); + sptr myObserver1 = sptr(new (std::nothrow) TestObserver()); Uri parentUri(info[1].uri); Uri newFileDir1(""); result = g_fah->Mkdir(parentUri, "uri_dir", newFileDir1); @@ -1246,7 +1247,7 @@ HWTEST_F(FileExtensionNotifyTest, external_file_access_notify_0020, testing::ext EXPECT_EQ(result, OHOS::FileAccessFwk::ERR_OK); bool notifyForDescendants = true; - sptr myObserver1 = new (std::nothrow) MyObserver(); + sptr myObserver1 = sptr(new (std::nothrow) MyObserver()); Uri parentUri(info[1].uri); Uri newFileDir1(""); SetNativeToken("SetUpTestCase", perms, sizeof(perms) / sizeof(perms[0])); diff --git a/test/unittest/file_access_service_proxy_test.cpp b/test/unittest/file_access_service_proxy_test.cpp index fc8ec7cf11cc2eb212c9434b429e57db58ff9d89..8446763035c77642fb44e4bcd7ae4104b6f8f76b 100644 --- a/test/unittest/file_access_service_proxy_test.cpp +++ b/test/unittest/file_access_service_proxy_test.cpp @@ -58,7 +58,7 @@ public: static void SetUpTestCase(void) { Assistant::ins_ = insMoc; - SystemAbilityManagerClient::GetInstance().systemAbilityManager_ = sptr(samgr); + SystemAbilityManagerClient::GetInstance().systemAbilityManager_ = sptr(samgr.get()); } static void TearDownTestCase() { @@ -73,7 +73,7 @@ public: public: static inline shared_ptr insMoc = make_shared(); static inline sptr impl = sptr(new FileAccessServiceMock()); - static inline ISystemAbilityManagerMock* samgr = new ISystemAbilityManagerMock(); + static inline shared_ptr samgr = make_shared(); }; /** diff --git a/test/unittest/medialibrary_file_access_test_basic.cpp b/test/unittest/medialibrary_file_access_test_basic.cpp index b31b0f3d0c12b57160e2c991c5b96c827d6c3f53..25c82ce4ee512c92c342ba7634c5a5c0842abeb4 100644 --- a/test/unittest/medialibrary_file_access_test_basic.cpp +++ b/test/unittest/medialibrary_file_access_test_basic.cpp @@ -43,17 +43,18 @@ const int UID_DEFAULT = 0; void SetNativeToken() { uint64_t tokenId; - const char **perms = new const char *[2]; + const char **perms = new const char *[3]; perms[0] = "ohos.permission.FILE_ACCESS_MANAGER"; perms[1] = "ohos.permission.GET_BUNDLE_INFO_PRIVILEGED"; + perms[2] = "ohos.permission.CONNECT_FILE_ACCESS_EXTENSION"; NativeTokenInfoParams infoInstance = { - .aplStr = "system_core", + .dcapsNum = 0, + .permsNum = 3, .aclsNum = 0, .dcaps = nullptr, - .dcapsNum = 0, - .permsNum = 2, .perms = perms, .acls = nullptr, + .aplStr = "system_core", }; infoInstance.processName = "SetUpTestCase"; diff --git a/test/unittest/mock/service_registry_mock.cpp b/test/unittest/mock/service_registry_mock.cpp index eecb89ed2a6b942c85e2cb010f214808395c8491..00c82437b76708560e8fd9af8d1f3926e2c7bdeb 100644 --- a/test/unittest/mock/service_registry_mock.cpp +++ b/test/unittest/mock/service_registry_mock.cpp @@ -25,7 +25,7 @@ namespace OHOS { using namespace OHOS::FileAccessFwk; SystemAbilityManagerClient& SystemAbilityManagerClient::GetInstance() { - static auto instance = new SystemAbilityManagerClient(); + static auto instance = std::make_shared(); return *instance; }