From a5b57e28ca4b3138cf655224e2853677f5dc9f87 Mon Sep 17 00:00:00 2001
From: lovechinamo <wangdongqi2@huawei.com>
Date: Thu, 2 Jun 2022 15:04:40 +0800
Subject: [PATCH] Signed-off-by: lovechinamo <wangdongqi2@huawei.com>

 Changes to be committed:
---
 .../src/input_method_core_proxy.cpp              |  2 +-
 .../src/input_method_system_ability_proxy.cpp    |  4 ++--
 services/src/peruser_setting.cpp                 | 16 ++++++++--------
 3 files changed, 11 insertions(+), 11 deletions(-)

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;
         }