diff --git a/interfaces/innerkits/service_control/service_control.c b/interfaces/innerkits/service_control/service_control.c
index fec0d09dd3e15217adc7ae44cfa80401c67e87dd..254bf5163599f38aa2094088a735513dfa97a651 100755
--- a/interfaces/innerkits/service_control/service_control.c
+++ b/interfaces/innerkits/service_control/service_control.c
@@ -166,7 +166,7 @@ int ServiceControlWithExtra(const char *serviceName, int action, const char *ext
 
 int ServiceControl(const char *serviceName, int action)
 {
-    if (serviceName == 0) {
+    if (serviceName == NULL) {
         HILOG_ERROR(LOG_CORE, "Service getctl failed, service is null.\n");
         return -1;
     }
diff --git a/ueventd/ueventd_device_handler.c b/ueventd/ueventd_device_handler.c
index 0e100c6b7a72ff20f7073ec6a2b859c7e52ef26a..324a570a42bd8ea230dcc18ca4852486ffff64a9 100755
--- a/ueventd/ueventd_device_handler.c
+++ b/ueventd/ueventd_device_handler.c
@@ -175,8 +175,16 @@ static char *FindPlatformDeviceName(char *path)
 
 static void BuildBootDeviceSymbolLink(char **links, int linkNum, const char *partitionName)
 {
+    if (links == NULL) {
+        INIT_LOGE("Function parameter error.");
+        return;
+    }
     if (linkNum > BLOCKDEVICE_LINKS - 1) {
-        INIT_LOGW("Too many links, ignore");
+        INIT_LOGW("Too many links, ignore.");
+        return;
+    }
+    if (partitionName == NULL) {
+        INIT_LOGW("Partition name is null, skip creating links");
         return;
     }
     links[linkNum] = calloc(sizeof(char), DEVICE_FILE_SIZE);