diff --git a/services/accesstokenmanager/main/cpp/src/database/access_token_open_callback.cpp b/services/accesstokenmanager/main/cpp/src/database/access_token_open_callback.cpp index fb5c7459f4efc61d05f3285d6d5a8b55e0b5cb37..f675982eff9011c94a35980711f1b571f75c9975 100644 --- a/services/accesstokenmanager/main/cpp/src/database/access_token_open_callback.cpp +++ b/services/accesstokenmanager/main/cpp/src/database/access_token_open_callback.cpp @@ -518,9 +518,8 @@ int32_t AccessTokenOpenCallback::OnUpgrade(NativeRdb::RdbStore& rdbStore, int32_ } case DATABASE_VERSION_2: // 2->3 - res = CreatePermissionRequestToggleStatusTable(rdbStore); + res = CreateVersionThreeTable(rdbStore); if (res != NativeRdb::E_OK) { - LOGE(ATM_DOMAIN, ATM_TAG, "Failed to create table permission_request_toggle_status_table."); return res; } diff --git a/services/accesstokenmanager/main/cpp/src/database/rdb_dlopen_manager.cpp b/services/accesstokenmanager/main/cpp/src/database/rdb_dlopen_manager.cpp index f423906b833c0f69fd0c1420478fccd03354ae7a..e95a4fd05c70b1f6f7ffbc059b01b6f7786f6631 100644 --- a/services/accesstokenmanager/main/cpp/src/database/rdb_dlopen_manager.cpp +++ b/services/accesstokenmanager/main/cpp/src/database/rdb_dlopen_manager.cpp @@ -186,9 +186,9 @@ int32_t RdbDlopenManager::Modify(const AtmDataType type, const GenericValues& mo return AccessTokenError::ERR_LOAD_SO_FAILED; } - taskNum_++; + ++taskNum_; int32_t res = instance->Modify(type, modifyValue, conditionValue); - taskNum_--; + --taskNum_; return res; } @@ -200,9 +200,9 @@ int32_t RdbDlopenManager::Find(AtmDataType type, const GenericValues& conditionV return AccessTokenError::ERR_LOAD_SO_FAILED; } - taskNum_++; + ++taskNum_; int32_t res = instance->Find(type, conditionValue, results); - taskNum_--; + --taskNum_; return res; } @@ -214,9 +214,9 @@ int32_t RdbDlopenManager::DeleteAndInsertValues(const std::vector& delI return AccessTokenError::ERR_LOAD_SO_FAILED; } - taskNum_++; + ++taskNum_; int32_t res = instance->DeleteAndInsertValues(delInfoVec, addInfoVec); - taskNum_--; + --taskNum_; return res; } } // namespace AccessToken diff --git a/services/accesstokenmanager/test/database_dlopen/database_dlopen_coverage_test.cpp b/services/accesstokenmanager/test/database_dlopen/database_dlopen_coverage_test.cpp index 24fafb20078371e036bb7def7f0610a18610fa0a..ea42a01544c91ce3f390be114b2a34180410719c 100644 --- a/services/accesstokenmanager/test/database_dlopen/database_dlopen_coverage_test.cpp +++ b/services/accesstokenmanager/test/database_dlopen/database_dlopen_coverage_test.cpp @@ -199,28 +199,6 @@ HWTEST_F(AccessTokenDatabaseDlopenTest, DelayDlcloseHandle002, TestSize.Level4) ASSERT_EQ(nullptr, instance->handle_); } -/* - * @tc.name: GetDbInstance001 - * @tc.desc: RdbDlopenManager::GetDbInstance - * @tc.type: FUNC - * @tc.require: - */ -HWTEST_F(AccessTokenDatabaseDlopenTest, GetDbInstance001, TestSize.Level4) -{ - auto instance = RdbDlopenManager::GetInstance(); - ASSERT_NE(nullptr, instance); - ASSERT_EQ(nullptr, instance->handle_); - ASSERT_EQ(nullptr, instance->instance_); - - instance->GetDbInstance(); // handle_ is nullptr - ASSERT_NE(nullptr, instance->handle_); - ASSERT_NE(nullptr, instance->instance_); - instance->GetDbInstance(); // handle_ is nullptr - sleep(WAIT_DELAY_DLCLOSE_TIME); - ASSERT_EQ(nullptr, instance->handle_); - ASSERT_EQ(nullptr, instance->instance_); -} - /* * @tc.name: DynamicCallTest001 * @tc.desc: RdbDlopenManager::DynamicCallTest diff --git a/services/accesstokenmanager/test/unittest/accesstoken_info_manager_test.cpp b/services/accesstokenmanager/test/unittest/accesstoken_info_manager_test.cpp index 5522bb9be66463e4e771d959bf452c70abbb603e..98bfddb261566f40dfc36338f62c531b9a67a8d9 100644 --- a/services/accesstokenmanager/test/unittest/accesstoken_info_manager_test.cpp +++ b/services/accesstokenmanager/test/unittest/accesstoken_info_manager_test.cpp @@ -2295,7 +2295,7 @@ HWTEST_F(AccessTokenInfoManagerTest, Dlopen001, TestSize.Level0) LibraryLoader loader1("libnotexist.z.so"); // is a not exist path EXPECT_EQ(nullptr, loader1.handle_); - LibraryLoader loader2("libaccesstoken_manager_service.z.so"); // is a exist lib without create func + LibraryLoader loader2("libaccesstoken_sdk.z.so"); // is a exist lib without create func EXPECT_EQ(nullptr, loader2.instance_); EXPECT_NE(nullptr, loader2.handle_); } diff --git a/services/privacymanager/test/unittest/permission_record_manager_test.cpp b/services/privacymanager/test/unittest/permission_record_manager_test.cpp index 0044434d087bd44c8347053447b17d75ad2df926..17877d3af072e3e9aca00e7bff14982b86711b38 100644 --- a/services/privacymanager/test/unittest/permission_record_manager_test.cpp +++ b/services/privacymanager/test/unittest/permission_record_manager_test.cpp @@ -934,22 +934,6 @@ HWTEST_F(PermissionRecordManagerTest, GetPermissionUsedType001, TestSize.Level0) ASSERT_EQ(0, PermissionRecordManager::GetInstance().GetPermissionUsedTypeInfos(tokenId, permissionName, results)); } -/** - * @tc.name: Dlopen001 - * @tc.desc: Open a not exist lib & not exist func - * @tc.type: FUNC - * @tc.require: - */ -HWTEST_F(PermissionRecordManagerTest, Dlopen001, TestSize.Level0) -{ - LibraryLoader loader1("libnotexist.z.so"); // is a not exist path - EXPECT_EQ(nullptr, loader1.handle_); - - LibraryLoader loader2("libaccesstoken_manager_service.z.so"); // is a exist lib without create func - EXPECT_EQ(nullptr, loader2.instance_); - EXPECT_NE(nullptr, loader2.handle_); -} - /* * @tc.name: AddDataValueToResults001 * @tc.desc: PermissionRecordManager::AddDataValueToResults function test