diff --git a/commondependency/src/deviceprofile_connector.cpp b/commondependency/src/deviceprofile_connector.cpp
index 66a809a343eb9bad4588774fa4c9b8662961f28a..2d7890f6669c56bc0fd00fd7db859137fd4a8452 100644
--- a/commondependency/src/deviceprofile_connector.cpp
+++ b/commondependency/src/deviceprofile_connector.cpp
@@ -1621,19 +1621,23 @@ void DeviceProfileConnector::HandleSyncBackgroundUserIdEvent(const std::vector<i
         std::string accesseeDeviceId = item.GetAccessee().GetAccesseeDeviceId();
         int32_t accesserUserId = item.GetAccesser().GetAccesserUserId();
         int32_t accesseeUserId = item.GetAccessee().GetAccesseeUserId();
-        if (accesserDeviceId == localUdid && accesseeDeviceId == remoteUdid && item.GetStatus() == ACTIVE &&
+        if (accesserDeviceId == localUdid && accesseeDeviceId == remoteUdid &&
             (find(remoteUserIds.begin(), remoteUserIds.end(), accesseeUserId) != remoteUserIds.end() ||
             find(localUserIds.begin(), localUserIds.end(), accesserUserId) == localUserIds.end())) {
-            item.SetStatus(INACTIVE);
-            DistributedDeviceProfileClient::GetInstance().UpdateAccessControlProfile(item);
+            if (item.GetStatus() == ACTIVE) {
+                item.SetStatus(INACTIVE);
+                DistributedDeviceProfileClient::GetInstance().UpdateAccessControlProfile(item);
+            }
             if (item.GetBindType() != DM_IDENTICAL_ACCOUNT) {
                 DistributedDeviceProfileClient::GetInstance().DeleteAccessControlProfile(item.GetAccessControlId());
             }
-        } else if ((accesseeDeviceId == localUdid && accesserDeviceId == remoteUdid) && item.GetStatus() == ACTIVE &&
+        } else if ((accesseeDeviceId == localUdid && accesserDeviceId == remoteUdid) &&
             (find(remoteUserIds.begin(), remoteUserIds.end(), accesserUserId) != remoteUserIds.end() ||
             find(localUserIds.begin(), localUserIds.end(), accesseeUserId) == localUserIds.end())) {
-            item.SetStatus(INACTIVE);
-            DistributedDeviceProfileClient::GetInstance().UpdateAccessControlProfile(item);
+            if (item.GetStatus() == ACTIVE) {
+                item.SetStatus(INACTIVE);
+                DistributedDeviceProfileClient::GetInstance().UpdateAccessControlProfile(item);
+            }
             if (item.GetBindType() != DM_IDENTICAL_ACCOUNT) {
                 DistributedDeviceProfileClient::GetInstance().DeleteAccessControlProfile(item.GetAccessControlId());
             }
@@ -1826,10 +1830,12 @@ bool DeviceProfileConnector::CheckAclStatusNotMatch(const DistributedDeviceProfi
 {
     if ((profile.GetAccesser().GetAccesserDeviceId() == localUdid &&
         (find(backgroundUserIds.begin(), backgroundUserIds.end(), profile.GetAccesser().GetAccesserUserId()) !=
-        backgroundUserIds.end()) && profile.GetStatus() == ACTIVE) ||
+        backgroundUserIds.end()) &&
+        (profile.GetStatus() == ACTIVE || profile.GetBindType() != DM_IDENTICAL_ACCOUNT)) ||
         (profile.GetAccessee().GetAccesseeDeviceId() == localUdid &&
         (find(backgroundUserIds.begin(), backgroundUserIds.end(), profile.GetAccessee().GetAccesseeUserId()) !=
-        backgroundUserIds.end()) && profile.GetStatus() == ACTIVE)) {
+        backgroundUserIds.end()) &&
+        (profile.GetStatus() == ACTIVE || profile.GetBindType() != DM_IDENTICAL_ACCOUNT))) {
         return true;
     }
     if ((profile.GetAccesser().GetAccesserDeviceId() == localUdid &&
@@ -1868,18 +1874,24 @@ int32_t DeviceProfileConnector::HandleUserSwitched(const std::string &localUdid,
     std::vector<AccessControlProfile> profiles = GetAllAccessControlProfile();
     std::vector<AccessControlProfile> activeProfiles;
     std::vector<AccessControlProfile> inActiveProfiles;
+    std::vector<AccessControlProfile> delActiveProfiles;
     for (auto &item : profiles) {
         if (std::find(deviceVec.begin(), deviceVec.end(), item.GetTrustDeviceId()) == deviceVec.end()) {
             continue;
         }
         if ((item.GetAccesser().GetAccesserDeviceId() == localUdid &&
             (find(backgroundUserIds.begin(), backgroundUserIds.end(),
-            item.GetAccesser().GetAccesserUserId()) != backgroundUserIds.end()) && item.GetStatus() == ACTIVE) ||
+            item.GetAccesser().GetAccesserUserId()) != backgroundUserIds.end())) ||
             (item.GetAccessee().GetAccesseeDeviceId() == localUdid &&
             (find(backgroundUserIds.begin(), backgroundUserIds.end(),
-            item.GetAccessee().GetAccesseeUserId()) != backgroundUserIds.end()) && item.GetStatus() == ACTIVE)) {
-            item.SetStatus(INACTIVE);
-            inActiveProfiles.push_back(item);
+            item.GetAccessee().GetAccesseeUserId()) != backgroundUserIds.end()))) {
+            if (item.GetStatus() == ACTIVE) {
+                item.SetStatus(INACTIVE);
+                inActiveProfiles.push_back(item);
+            }
+            if (item.GetBindType() != DM_IDENTICAL_ACCOUNT) {
+                delActiveProfiles.push_back(item);
+            }
             continue;
         }
         if ((item.GetAccesser().GetAccesserDeviceId() == localUdid &&
@@ -1893,15 +1905,7 @@ int32_t DeviceProfileConnector::HandleUserSwitched(const std::string &localUdid,
             continue;
         }
     }
-    for (auto &item : inActiveProfiles) {
-        DistributedDeviceProfileClient::GetInstance().UpdateAccessControlProfile(item);
-        if (item.GetBindType() != DM_IDENTICAL_ACCOUNT) {
-            DistributedDeviceProfileClient::GetInstance().DeleteAccessControlProfile(item.GetAccessControlId());
-        }
-    }
-    for (auto &item : activeProfiles) {
-        DistributedDeviceProfileClient::GetInstance().UpdateAccessControlProfile(item);
-    }
+    HandleUserSwitched(activeProfiles, inActiveProfiles, delActiveProfiles);
     return DM_OK;
 }
 
diff --git a/services/service/src/device_manager_service.cpp b/services/service/src/device_manager_service.cpp
index 493f45f5536f3ec6aaa61f1cfd2809021f8181d8..7c9108569d12e935e7d162c701d49c1676ca7237 100644
--- a/services/service/src/device_manager_service.cpp
+++ b/services/service/src/device_manager_service.cpp
@@ -2209,7 +2209,7 @@ void DeviceManagerService::HandleUserIdsBroadCast(const std::vector<UserIdInfo>
         int32_t retFront = MultipleUserConnector::GetForegroundUserIds(foregroundUserVec);
         int32_t retBack = MultipleUserConnector::GetBackgroundUserIds(backgroundUserVec);
         if (IsPC()) {
-            MultipleUserConnector::ClearLockedUser(foregroundUserVec, backgroundUserVec);
+            MultipleUserConnector::ClearLockedUser(foregroundUserVec);
         }
         if (retFront != DM_OK || retBack!= DM_OK) {
             LOGE("Get userid failed, retFront: %{public}d, retBack: %{public}d, frontUserNum:%{public}d,"
diff --git a/services/service/src/relationshipsyncmgr/dm_comm_tool.cpp b/services/service/src/relationshipsyncmgr/dm_comm_tool.cpp
index 01a64b6723649cd6fa12b21627f7e295a564f09f..5fb82c255f4a937c425eeb066d3f637589184f95 100644
--- a/services/service/src/relationshipsyncmgr/dm_comm_tool.cpp
+++ b/services/service/src/relationshipsyncmgr/dm_comm_tool.cpp
@@ -213,7 +213,7 @@ void DMCommTool::ProcessReceiveUserIdsEvent(const std::shared_ptr<InnerCommMsg>
     MultipleUserConnector::GetForegroundUserIds(foregroundUserIds);
     MultipleUserConnector::GetBackgroundUserIds(backgroundUserIds);
     if (DeviceManagerService::GetInstance().IsPC()) {
-        MultipleUserConnector::ClearLockedUser(foregroundUserIds, backgroundUserIds);
+        MultipleUserConnector::ClearLockedUser(foregroundUserIds);
     }
     std::vector<uint32_t> foregroundUserIdsU32;
     std::vector<uint32_t> backgroundUserIdsU32;