init: fix codedex

Signed-off-by: Nxionglei6 <xionglei6@huawei.com>

services/begetctl/main.c

@@ -28,13 +28,11 @@ static struct CMD_LIST_ST *m_cmdList = NULL;
 
 int BegetCtlCmdAdd(const char *name, BegetCtlCmdPtr cmd)
 {
-    struct CMD_LIST_ST *item;
-
     if (name == NULL) {
         return -1;
     }
 
-    item = (struct CMD_LIST_ST *)malloc(sizeof(struct CMD_LIST_ST));
+    struct CMD_LIST_ST *item = (struct CMD_LIST_ST *)malloc(sizeof(struct CMD_LIST_ST));
     if (item == NULL) {
         return -1;
     }
@@ -83,7 +81,6 @@ static void BegetCtlUsage(const char *command)
 
 int main(int argc, char **argv)
 {
-    const struct CMD_LIST_ST *cmd;
     const char *last = strrchr(argv[0], '/');
 
     // Get the first ending command name
@@ -101,7 +98,7 @@ int main(int argc, char **argv)
     }
 
     // Match the command
-    cmd = BegetCtlCmdFind(last);
+    const struct CMD_LIST_ST *cmd = BegetCtlCmdFind(last);
     if (cmd == NULL) {
         BegetCtlUsage(last);
         return 0;

services/init/standard/init_reboot.c

@@ -48,16 +48,16 @@ static int RBMiscWriteUpdaterMessage(const char *path, const struct RBMiscUpdate
     INIT_CHECK_RETURN_VALUE(realPath != NULL, -1);
     int ret = 0;
     FILE *fp = fopen(realPath, "rb+");
+    free(realPath);
+    realPath = NULL;
     if (fp != NULL) {
         size_t writeLen = fwrite(boot, sizeof(struct RBMiscUpdateMessage), 1, fp);
         INIT_ERROR_CHECK(writeLen == 1, ret = -1, "Failed to write misc for reboot");
+        (void)fclose(fp);
     } else {
         ret = -1;
         INIT_LOGE("Failed to open %s", path);
     }
-    free(realPath);
-    realPath = NULL;
-    (void)fclose(fp);
     return ret;
 }
 

test/unittest/init/cmds_unittest.cpp

@@ -103,7 +103,7 @@ HWTEST_F(CmdsUnitTest, TestCommonChmod, TestSize.Level1)
 {
     const char *testFile = "/data/init_ut/test_dir0/test_file0";
     const mode_t testMode = S_IRWXU | S_IRWXG | S_IRWXO;
-    int fd = open(testFile, O_CREAT | O_WRONLY);
+    int fd = open(testFile, O_CREAT | O_WRONLY, testMode);
     ASSERT_GE(fd, 0);
     DoCmdByName("chmod ", "777 /data/init_ut/test_dir0/test_file0");
     struct stat info;

ueventd/ueventd.c

@@ -133,13 +133,13 @@ static void HandleUeventRequired(const struct Uevent *uevent, char **devices, in
             if (uevent->partitionName == NULL) {
                 INIT_LOGI("Match with %s for %s", devices[i], uevent->syspath);
                 deviceName = strstr(devices[i], "/dev/block");
-                if (deviceName != NULL) {
-                    deviceName += sizeof("/dev/block") - 1;
-                    if (strstr(uevent->syspath, deviceName) != NULL) {
-                        HandleBlockDeviceEvent(uevent);
-                        break;
-                    }
-                }
+                INIT_INFO_CHECK(deviceName != NULL, continue,
+                    "device %s not match \"/dev/block\".", devices[i]);
+                deviceName += sizeof("/dev/block") - 1;
+                INIT_INFO_CHECK(strstr(uevent->syspath, deviceName) != NULL, continue,
+                    "uevent->syspath %s not match deviceName %s", uevent->syspath, deviceName);
+                HandleBlockDeviceEvent(uevent);
+                break;
             } else if (strstr(devices[i], uevent->partitionName) != NULL) {
                 INIT_LOGI("Handle block device partitionName %s", uevent->partitionName);
                 HandleBlockDeviceEvent(uevent);