diff --git a/frameworks/inputmethod_ability/src/input_method_core_proxy.cpp b/frameworks/inputmethod_ability/src/input_method_core_proxy.cpp
index e25cd1c..1626863 100644
--- a/frameworks/inputmethod_ability/src/input_method_core_proxy.cpp
+++ b/frameworks/inputmethod_ability/src/input_method_core_proxy.cpp
@@ -308,7 +308,7 @@ namespace MiscServices {
         if (code) {
             return code;
         }
-        *retHeight = reply.ReadInt32();
+        retHeight = reply.ReadInt32();
         return ErrorCode::NO_ERROR;
     }
 } // namespace MiscServices
diff --git a/frameworks/inputmethod_controller/src/input_method_system_ability_proxy.cpp b/frameworks/inputmethod_controller/src/input_method_system_ability_proxy.cpp
index 3924212..c639b6a 100644
--- a/frameworks/inputmethod_controller/src/input_method_system_ability_proxy.cpp
+++ b/frameworks/inputmethod_controller/src/input_method_system_ability_proxy.cpp
@@ -274,7 +274,7 @@ namespace MiscServices {
             return ret;
         }
 
-        if (!reply.ReadInt32(*retMode)) {
+        if (!reply.ReadInt32(retMode)) {
             return ERROR_STATUS_BAD_VALUE;
         }
         return NO_ERROR;
@@ -299,7 +299,7 @@ namespace MiscServices {
             return ret;
         }
 
-        if (!reply.ReadInt32(*retHeight)) {
+        if (!reply.ReadInt32(retHeight)) {
             return ERROR_STATUS_BAD_VALUE;
         }
         return NO_ERROR;
diff --git a/services/src/peruser_setting.cpp b/services/src/peruser_setting.cpp
index ba74239..2fc1d64 100644
--- a/services/src/peruser_setting.cpp
+++ b/services/src/peruser_setting.cpp
@@ -78,8 +78,8 @@ namespace MiscServices {
      */
     int PerUserSetting::OnPackageAdded(std::u16string& packageName, bool &isSecurityIme)
     {
-        if (*isSecurityIme) {
-            *isSecurityIme = false;
+        if (isSecurityIme) {
+            isSecurityIme = false;
         }
         std::u16string imeId = GetImeId(packageName);
         if (imeId.size()) {
@@ -97,8 +97,8 @@ namespace MiscServices {
         }
         inputMethodProperties.push_back(property);
         if (CheckIfSecurityIme(*property)) {
-            if (*isSecurityIme) {
-                *isSecurityIme = true;
+            if (isSecurityIme) {
+                isSecurityIme = true;
             }
             return ErrorCode::NO_ERROR;
         }
@@ -128,8 +128,8 @@ namespace MiscServices {
      */
     int PerUserSetting::OnPackageRemoved(std::u16string& packageName, bool &isSecurityIme)
     {
-        if (*isSecurityIme) {
-            *isSecurityIme = false;
+        if (isSecurityIme) {
+            isSecurityIme = false;
         }
         std::u16string imeId = GetImeId(packageName);
         if (!imeId.size()) {
@@ -151,8 +151,8 @@ namespace MiscServices {
             }
         }
         if (securityFlag) {
-            if (*isSecurityIme) {
-                *isSecurityIme = true;
+            if (isSecurityIme) {
+                isSecurityIme = true;
             }
             return ErrorCode::NO_ERROR;
         }