diff --git a/services/param/include/param_manager.h b/services/param/include/param_manager.h
index e2aeeb931a2d09417c70f2e4c307e0d29556b979..ec63e7a8a3ecec7697307146849f5ee1c7ca9df6 100644
--- a/services/param/include/param_manager.h
+++ b/services/param/include/param_manager.h
@@ -152,7 +152,7 @@ INIT_LOCAL_API void ClosePersistParamWorkSpace(void);
 INIT_LOCAL_API int WritePersistParam(const char *name, const char *value);
 
 INIT_LOCAL_API int CheckParameterSet(const char *name, const char *value,
-    const ParamSecurityLabel *srcLabel, int *ctrlService);
+    const ParamSecurityLabel *srcLabel, uint32_t *ctrlService);
 
 INIT_LOCAL_API int CheckParamPermission(const ParamSecurityLabel *srcLabel, const char *name, uint32_t mode);
 
diff --git a/services/param/linux/param_service.c b/services/param/linux/param_service.c
index 9332af772e72c2c43a30d465faf82b8715e0889c..56792f4d04ab1c9af2ad4ee7b34568d17e055dbc 100755
--- a/services/param/linux/param_service.c
+++ b/services/param/linux/param_service.c
@@ -139,7 +139,7 @@ static int SendWatcherNotifyMessage(const TriggerExtInfo *extData, const char *c
 static int SystemSetParam(const char *name, const char *value, const ParamSecurityLabel *srcLabel)
 {
     PARAM_LOGV("SystemWriteParam name %s value: %s", name, value);
-    int ctrlService = 0;
+    uint32_t ctrlService = 0;
     int ret = CheckParameterSet(name, value, srcLabel, &ctrlService);
     PARAM_CHECK(ret == 0, return ret, "Forbid to set parameter %s", name);
 
@@ -162,7 +162,7 @@ static int SystemUpdateConstParamter(const char *name, const char *value, const
 {
     PARAM_LOGV("SystemUpdateConstParam name %s value: %s", name, value);
     uint32_t ctrlService = 0;
-    int ret = CheckParameterSet(name, value, srcLabel, (int *)&ctrlService);
+    int ret = CheckParameterSet(name, value, srcLabel, &ctrlService);
     PARAM_CHECK(ret == 0, return ret, "Forbid to set parameter %s", name);
     PARAM_CHECK(IS_READY_ONLY(name), return PARAM_CODE_INVALID_NAME, "only update read only param: %s", name);
     if ((ctrlService & PARAM_CTRL_SERVICE) != PARAM_CTRL_SERVICE) { // ctrl param
diff --git a/services/param/liteos/param_client.c b/services/param/liteos/param_client.c
index 4569fd8a2c0af50561bc65e4a3235ddda55c3a8a..4b9179ff4af86a976394d003d659bd454c55840b 100644
--- a/services/param/liteos/param_client.c
+++ b/services/param/liteos/param_client.c
@@ -48,7 +48,7 @@ __attribute__((destructor)) static void ClientDeinit(void)
 int SystemSetParameter(const char *name, const char *value)
 {
     PARAM_CHECK(name != NULL && value != NULL, return -1, "Invalid name or value %s", name);
-    int ctrlService = 0;
+    uint32_t ctrlService = 0;
     int ret = CheckParameterSet(name, value, GetParamSecurityLabel(), &ctrlService);
     PARAM_CHECK(ret == 0, return ret, "Forbid to set parameter %s", name);
     PARAM_LOGV("SystemSetParameter name %s value: %s ctrlService %d", name, value, ctrlService);
@@ -72,7 +72,7 @@ int SystemWaitParameter(const char *name, const char *value, int32_t timeout)
     // first check permission
     int ret = CheckParamPermission(GetParamSecurityLabel(), name, DAC_READ);
     PARAM_CHECK(ret == 0, return ret, "SystemWaitParameter failed! name is: %s, errNum is: %d", name, ret);
-    uint32_t diff = 0;
+    int32_t diff = 0;
     struct timespec startTime = {0};
     if (timeout <= 0) {
         timeout = DEFAULT_PARAM_WAIT_TIMEOUT;
diff --git a/services/param/liteos/param_service.c b/services/param/liteos/param_service.c
index cd7f5d9d3847c767b6d90ea803e14134d854e418..7c8c899d5effa7a7dbe6d601af2c1d04f4a96725 100644
--- a/services/param/liteos/param_service.c
+++ b/services/param/liteos/param_service.c
@@ -103,7 +103,7 @@ void StopParamService(void)
 
 int SystemWriteParam(const char *name, const char *value)
 {
-    int ctrlService = 0;
+    uint32_t ctrlService = 0;
     int ret = CheckParameterSet(name, value, GetParamSecurityLabel(), &ctrlService);
     PARAM_CHECK(ret == 0, return ret, "Forbid to set parameter %s", name);
     PARAM_LOGV("SystemWriteParam name %s value: %s ctrlService %d", name, value, ctrlService);
diff --git a/services/param/manager/param_manager.c b/services/param/manager/param_manager.c
index 10e7061e5e550997d9946c59af9be8e5da3eb424..067c3458281560f26925af5aaf027bffd3379a03 100644
--- a/services/param/manager/param_manager.c
+++ b/services/param/manager/param_manager.c
@@ -376,7 +376,7 @@ INIT_LOCAL_API int GetServiceCtrlInfo(const char *name, const char *value, Servi
 }
 
 INIT_LOCAL_API int CheckParameterSet(const char *name,
-    const char *value, const ParamSecurityLabel *srcLabel, int *ctrlService)
+    const char *value, const ParamSecurityLabel *srcLabel, uint32_t *ctrlService)
 {
     ParamWorkSpace *paramSpace = GetParamWorkSpace();
     PARAM_CHECK(paramSpace != NULL, return PARAM_WORKSPACE_NOT_INIT, "Invalid paramSpace");