diff --git a/apps/lms/src/sample_usr_lms.c b/apps/lms/src/sample_usr_lms.c
index c8ef15d63c989ababf52d1158909d12e159140d0..00a861181c021f5ab6116ff4f4a23faeb6f5faa7 100644
--- a/apps/lms/src/sample_usr_lms.c
+++ b/apps/lms/src/sample_usr_lms.c
@@ -180,7 +180,8 @@ static void LmsStrcpyTest(void)
         return;
     }
     char *testStr = "bbbbbbbbbbbbbbbbb";
-    printf("[LmsStrcpyTest] strcpy overflow error should be triggered, src string buf size:%d\n", strlen(testStr) + 1);
+    printf("[LmsStrcpyTest] strcpy overflow error should be triggered, src string buf size:%d\n",
+           (int)strlen(testStr) + 1);
     strcpy(buf, testStr);
     free(buf);
     printf("\n-------- LmsStrcpyTest End --------\n");
diff --git a/apps/perf/src/main.c b/apps/perf/src/main.c
index 8fb841fa8592a0f609d171cda468594622c7fbb8..1482d490cfa93f47cab6c9bd883c05c780887ca7 100644
--- a/apps/perf/src/main.c
+++ b/apps/perf/src/main.c
@@ -62,7 +62,7 @@ int main(int argc, char **argv)
         PerfStop(fd);
     } else if ((argc == THREE_ARGS) && strcmp(argv[1], "read") == 0) {
         size_t size = strtoul(argv[THREE_ARGS - 1], NULL, 0);
-        if (size <= 0) {
+        if (size == 0) {
             goto EXIT:
         }
 
diff --git a/apps/tftp/src/tftpc.c b/apps/tftp/src/tftpc.c
index bc67cf48ecb57d14e97a15ca73333960607be47d..6185c2af594d3061a655becc0439cc609c67357f 100644
--- a/apps/tftp/src/tftpc.c
+++ b/apps/tftp/src/tftpc.c
@@ -1437,10 +1437,8 @@ u32_t lwip_tftp_get_file_by_filename_to_rawmem(u32_t ulHostAddr,
 {
     s32_t iSockNum = TFTP_NULL_INT32;
     u32_t ulSrcStrLen;
-    u32_t lDestStrLen;
     u32_t ulSize;
     u32_t ulRecvSize = TFTP_NULL_UINT32;
-    s32_t iErrCode;
     u32_t ulErrCode;
     u16_t usReadReq;
     u16_t usTempServPort;
diff --git a/apps/trace/src/trace.c b/apps/trace/src/trace.c
index ddcb555385184a7a13d9e1fc0dcdd1347fc9649d..ecc4f22941245bd9a08dc815b3d999f6f6cb8ec0 100644
--- a/apps/trace/src/trace.c
+++ b/apps/trace/src/trace.c
@@ -70,7 +70,7 @@ static void TraceRead(int fd, size_t size)
 {
     ssize_t i;
     ssize_t len;
-    if (size <= 0) {
+    if (size == 0) {
         return;
     }
 
diff --git a/fs/fat/os_adapt/fatfs.c b/fs/fat/os_adapt/fatfs.c
index 3bb9f4895ff4e74f1205dd62bee84aa6f9a09bba..e95706a6e776f05ba877000e64fb95c90d5ded46 100644
--- a/fs/fat/os_adapt/fatfs.c
+++ b/fs/fat/os_adapt/fatfs.c
@@ -1409,7 +1409,7 @@ DWORD fattime_format(time_t time)
     ftime |= ((DWORD)((st.tm_year > YEAR_OFFSET) ? (st.tm_year - YEAR_OFFSET) : 0)) << FTIME_YEAR_OFFSET;
     ftime <<= FTIME_DATE_OFFSET;
 
-    ftime = (DWORD)st.tm_sec / SEC_MULTIPLIER;
+    ftime |= (DWORD)st.tm_sec / SEC_MULTIPLIER;
     ftime |= ((DWORD)st.tm_min) << FTIME_MIN_OFFSET;
     ftime |= ((DWORD)st.tm_hour) << FTIME_HR_OFFSET;
 
@@ -2167,7 +2167,7 @@ int fatfs_symlink(struct Vnode *parentVnode, struct Vnode **newVnode, const char
 ssize_t fatfs_readlink(struct Vnode *vnode, char *buffer, size_t bufLen)
 {
     int ret;
-    FRESULT res = FR_OK;
+    FRESULT res;
     DWORD clust;
     QWORD sect;
     DIR_FILE *dfp = (DIR_FILE *)(vnode->data);
diff --git a/fs/vfs/bcache/src/bcache.c b/fs/vfs/bcache/src/bcache.c
index c9e2d795d9ff63cddcb2c55405e48029f13c6f06..c1e763764c005be111300883edd2a0c15d6e527e 100644
--- a/fs/vfs/bcache/src/bcache.c
+++ b/fs/vfs/bcache/src/bcache.c
@@ -583,9 +583,6 @@ static OsBcacheBlock *AllocNewBlock(OsBcache *bc, BOOL read, UINT64 num)
 
     if (prefer->used) {
         MergeSyncBlocks(bc, prefer);
-    }
-
-    if (prefer->used) {
         BcacheSyncBlock(bc, prefer);
         DelBlock(bc, prefer);
     }
diff --git a/fs/vfs/include/vnode.h b/fs/vfs/include/vnode.h
index f04e1a11935824e21aaae28c76120cb25ce4e3bf..3c16a63368572632fa097713bfb3ab76d5a0c867 100644
--- a/fs/vfs/include/vnode.h
+++ b/fs/vfs/include/vnode.h
@@ -160,7 +160,7 @@ typedef int VfsHashCmp(struct Vnode *vnode, void *arg);
 
 int VnodesInit(void);
 int VnodeDevInit(void);
-int VnodeAlloc(struct VnodeOps *vop, struct Vnode **vnode);
+int VnodeAlloc(struct VnodeOps *vop, struct Vnode **newVnode);
 int VnodeFree(struct Vnode *vnode);
 int VnodeLookup(const char *path, struct Vnode **vnode, uint32_t flags);
 int VnodeLookupFullpath(const char *fullpath, struct Vnode **vnode, uint32_t flags);
diff --git a/fs/vfs/vfs_cmd/vfs_shellcmd.c b/fs/vfs/vfs_cmd/vfs_shellcmd.c
index a34f0d9ba9b559ce1333424404176c7040bd3179..6360555f76720939d67c8458fa9c2e53dee2250d 100644
--- a/fs/vfs/vfs_cmd/vfs_shellcmd.c
+++ b/fs/vfs/vfs_cmd/vfs_shellcmd.c
@@ -761,11 +761,6 @@ static int os_shell_cmd_do_rmdir(const char *pathname)
         }
         if (strcmp(dirent->d_name, "..") && strcmp(dirent->d_name, ".")) {
             size_t fullpath_buf_size = strlen(pathname) + strlen(dirent->d_name) + SEPARATOR_EOF_LEN;
-            if (fullpath_buf_size <= 0) {
-                PRINTK("buffer size is invalid!\n");
-                (void)closedir(d);
-                return -1;
-            }
             fullpath = (char *)malloc(fullpath_buf_size);
             if (fullpath == NULL) {
                 PRINTK("malloc failure!\n");
diff --git a/fs/vfs/vnode.c b/fs/vfs/vnode.c
index 41fb043f7867440fcc941e655037da147e4b2545..fb0ce33052b7052348cac630e9263c4a6a7bcd9d 100644
--- a/fs/vfs/vnode.c
+++ b/fs/vfs/vnode.c
@@ -260,7 +260,7 @@ int VnodeDrop(void)
 static char *NextName(char *pos, uint8_t *len)
 {
     char *name = NULL;
-    while (*pos != 0 && *pos == '/') {
+    while (*pos == '/') {
         pos++;
     }
     if (*pos == '\0') {
diff --git a/kernel/base/mp/los_lockdep.c b/kernel/base/mp/los_lockdep.c
index fbbbadfcbc704b7a9915c1baeda8304a749c8233..fb09359135a342d734ea88b3a32b6de97236ce99 100644
--- a/kernel/base/mp/los_lockdep.c
+++ b/kernel/base/mp/los_lockdep.c
@@ -271,7 +271,7 @@ VOID OsLockDepCheckOut(SPIN_LOCK_S *lock)
 {
     UINT32 intSave;
     INT32 depth;
-    enum LockDepErrType checkResult = LOCKDEP_SUCCESS;
+    enum LockDepErrType checkResult;
     VOID *requestAddr = (VOID *)__builtin_return_address(1);
     LosTaskCB *current = OsCurrTaskGet();
     LosTaskCB *owner = NULL;
diff --git a/kernel/base/vm/los_vm_dump.c b/kernel/base/vm/los_vm_dump.c
index 1bb804cdd41a3e11e789b2bba28d8ba0ff8a9675..14f6ed606eae7fe48d44661ff25e81cc72107e67 100644
--- a/kernel/base/vm/los_vm_dump.c
+++ b/kernel/base/vm/los_vm_dump.c
@@ -208,15 +208,12 @@ UINT32 OsUProcessPmUsage(LosVmSpace *space, UINT32 *sharePm, UINT32 *actualPm)
     PADDR_T paddr;
     STATUS_T ret;
     INT32 shareRef;
+    UINT32 pmSize = 0;
 
     if (sharePm != NULL) {
         *sharePm = 0;
     }
 
-    if (actualPm != NULL) {
-        *actualPm = 0;
-    }
-
     ret = LOS_MuxAcquire(&space->regionMux);
     if (ret != 0) {
         return 0;
@@ -240,19 +237,20 @@ UINT32 OsUProcessPmUsage(LosVmSpace *space, UINT32 *sharePm, UINT32 *actualPm)
                 if (sharePm != NULL) {
                     *sharePm += PAGE_SIZE;
                 }
-                if (actualPm != NULL) {
-                    *actualPm += PAGE_SIZE / shareRef;
-                }
+                pmSize += PAGE_SIZE / shareRef;
             } else {
-                if (actualPm != NULL) {
-                    *actualPm += PAGE_SIZE;
-                }
+                pmSize += PAGE_SIZE;
             }
         }
     RB_SCAN_SAFE_END(&oldVmSpace->regionRbTree, pstRbNode, pstRbNodeNext)
 
     (VOID)LOS_MuxRelease(&space->regionMux);
-    return *actualPm;
+
+    if (actualPm != NULL) {
+        *actualPm = pmSize;
+    }
+
+    return pmSize;
 }
 
 LosProcessCB *OsGetPIDByAspace(LosVmSpace *space)
@@ -305,7 +303,7 @@ UINT32 OsCountRegionPages(LosVmSpace *space, LosVmMapRegion *region, UINT32 *pss
     }
 
     if (pssPages != NULL) {
-        *pssPages = (UINT32)(pss + 0.5);
+        *pssPages = (UINT32)(pss + 0.5);    /* 0.5, for page alignment */
     }
 
     return regionPages;
diff --git a/kernel/base/vm/los_vm_scan.c b/kernel/base/vm/los_vm_scan.c
index 231ed18e77692526f293c73578446bdb2cdc7486..ab467818a2bd42ee81a336a80ebf6696f55200af 100644
--- a/kernel/base/vm/los_vm_scan.c
+++ b/kernel/base/vm/los_vm_scan.c
@@ -297,7 +297,7 @@ int OsTryShrinkMemory(size_t nPage)
     LosFilePage *fpage = NULL;
     LosFilePage *fnext = NULL;
 
-    if (nPage <= 0) {
+    if (nPage == 0) {
         nPage = VM_FILEMAP_MIN_SCAN;
     }
 
diff --git a/kernel/base/vm/los_vm_syscall.c b/kernel/base/vm/los_vm_syscall.c
index 1f6d600e8547fca6284f3c284bf7d2cd7d63e361..6aa2399317e39271ca4659f387e657c3eb2f6d9c 100644
--- a/kernel/base/vm/los_vm_syscall.c
+++ b/kernel/base/vm/los_vm_syscall.c
@@ -166,7 +166,7 @@ MMAP_DONE:
 
 STATUS_T LOS_UnMMap(VADDR_T addr, size_t size)
 {
-    if ((addr <= 0) || (size <= 0)) {
+    if ((addr <= 0) || (size == 0)) {
         return -EINVAL;
     }
 
diff --git a/kernel/common/console.c b/kernel/common/console.c
index 896d332b5c1388f5247eff24cbfa3e1755d770d1..800b43b9555802bf1b19c3814770bec571aa646b 100644
--- a/kernel/common/console.c
+++ b/kernel/common/console.c
@@ -709,7 +709,6 @@ STATIC ssize_t DoWrite(CirBufSendCB *cirBufSendCB, CHAR *buffer, size_t bufLen)
 {
     INT32 cnt;
     size_t written = 0;
-    size_t toWrite = bufLen;
     UINT32 intSave;
 
 #ifdef LOSCFG_SHELL_DMESG
@@ -730,7 +729,6 @@ STATIC ssize_t DoWrite(CirBufSendCB *cirBufSendCB, CHAR *buffer, size_t bufLen)
         if (cnt <= 0) {
             break;
         }
-        toWrite -= cnt;
         written += cnt;
     }
     LOS_SpinUnlockRestore(&g_consoleWriteSpinLock, intSave);
@@ -1536,7 +1534,15 @@ STATIC ssize_t WriteToTerminal(const CONSOLE_CB *consoleCB, const CHAR *buffer,
 
     fd = consoleCB->fd;
     ret = fs_getfilep(fd, &filep);
+    if (ret < 0) {
+        ret = -EPERM;
+        goto ERROUT;
+    }
     ret = GetFilepOps(filep, &privFilep, &fileOps);
+    if (ret != ENOERR) {
+        ret = -EINVAL;
+        goto ERROUT;
+    }
 
     if ((fileOps == NULL) || (fileOps->write == NULL)) {
         ret = EFAULT;
diff --git a/kernel/common/los_init.c b/kernel/common/los_init.c
index 408bace9354fe9eea37e6d40678b7a52b7e408c0..3b9c4b416c64a3b277afb08d9162372878bc9922 100644
--- a/kernel/common/los_init.c
+++ b/kernel/common/los_init.c
@@ -87,7 +87,6 @@ STATIC VOID InitLevelCall(const CHAR *name, const UINT32 level, struct ModuleIni
         LOS_SpinUnlock(&g_initLock);
         if (module->hook != NULL) {
 #ifdef LOS_INIT_DEBUG
-            ret = LOS_OK;
             startNsec = LOS_CurrNanosec();
             ret = (UINT32)module->hook();
             endNsec = LOS_CurrNanosec();
diff --git a/kernel/extended/dynload/src/los_load_elf.c b/kernel/extended/dynload/src/los_load_elf.c
index 65c2eeeeb5d927cd8d18cbf1f2c4f182c2eba6d6..af4c552763f10a91a2c4e8f410e26875597c1c7f 100644
--- a/kernel/extended/dynload/src/los_load_elf.c
+++ b/kernel/extended/dynload/src/los_load_elf.c
@@ -51,7 +51,7 @@ STATIC BOOL g_srandInit;
 
 STATIC INT32 OsELFOpen(const CHAR *fileName, INT32 oflags)
 {
-    INT32 ret = -LOS_NOK;
+    INT32 ret;
     INT32 procFd;
 
     procFd = AllocProcessFd();
diff --git a/kernel/extended/lms/usr/los_lms.c b/kernel/extended/lms/usr/los_lms.c
index 522f1f9c6d818f0effdca4b506db18302bf30f03..374a0f742edae4bfce480aeb187a9666255941e4 100644
--- a/kernel/extended/lms/usr/los_lms.c
+++ b/kernel/extended/lms/usr/los_lms.c
@@ -124,8 +124,8 @@ ATTRIBUTE_NO_SANITIZE_ADDRESS void LmsSetShadowValue(uintptr_t startAddr, uintpt
     uint32_t startOffset;
     uint32_t endOffset;
 
-    char shadowValueMask;
-    char shadowValue;
+    unsigned char shadowValueMask;
+    unsigned char shadowValue;
 
     /* endAddr - 1, then we mark [startAddr, endAddr) to value */
     LmsMem2Shadow(startAddr, &shadowStart, &startOffset);
@@ -266,7 +266,7 @@ ATTRIBUTE_NO_SANITIZE_ADDRESS void LmsPrintMemInfo(uintptr_t addr)
             }
         }
 
-        LMS_OUTPUT_INFO("|\t[0x%x | %2d]: ", shadowAddr, shadowOffset);
+        LMS_OUTPUT_INFO("|\t[0x%x | %2u]: ", shadowAddr, shadowOffset);
 
         for (int x = 0; x < printX; x += LMS_MEM_BYTES_PER_SHADOW_CELL) {
             LmsGetShadowValue(dumpAddr + x, &shadowValue);
@@ -339,7 +339,7 @@ ATTRIBUTE_NO_SANITIZE_ADDRESS static void LmsPrintErrInfo(LmsAddrInfo *info, uin
             break;
     }
 
-    LMS_OUTPUT_INFO("Shadow memory address: [0x%x : %d]  Shadow memory value: [%d] \n", info->shadowAddr,
+    LMS_OUTPUT_INFO("Shadow memory address: [0x%x : %u]  Shadow memory value: [%u] \n", info->shadowAddr,
         info->shadowOffset, info->shadowValue);
 
     LMS_OUTPUT_INFO("\n");
diff --git a/net/lwip-2.1/enhancement/src/api_shell.c b/net/lwip-2.1/enhancement/src/api_shell.c
index 58b532779d915828adc595b1370109f72afb9009..5c56d3652ea03825a15a05262284439787c2b328 100644
--- a/net/lwip-2.1/enhancement/src/api_shell.c
+++ b/net/lwip-2.1/enhancement/src/api_shell.c
@@ -342,7 +342,7 @@ int print_netif(struct netif *netif, char *print_buf, unsigned int buf_len)
     buf_len -= (unsigned int)ret;
 
     ret = snprintf_s(tmp, buf_len, (buf_len - 1), " MTU:%d %s", netif->mtu,
-                     netif->flags & NETIF_FLAG_UP ? "Running" : "Stop");
+                     (netif->flags & NETIF_FLAG_UP) ? "Running" : "Stop");
     if ((ret <= 0) || ((unsigned int)ret >= buf_len))
         goto out;
     tmp += ret;
@@ -356,7 +356,8 @@ int print_netif(struct netif *netif, char *print_buf, unsigned int buf_len)
         buf_len -= (unsigned int)ret;
     }
 
-    ret = snprintf_s(tmp, buf_len, (buf_len - 1), " %s\n", netif->flags & NETIF_FLAG_LINK_UP ? "Link UP" : "Link Down");
+    ret = snprintf_s(tmp, buf_len, (buf_len - 1), " %s\n", 
+                     (netif->flags & NETIF_FLAG_LINK_UP) ? "Link UP" : "Link Down");
     if ((ret <= 0) || ((unsigned int)ret >= buf_len))
         goto out;
     tmp += ret;
@@ -3019,8 +3020,8 @@ int netstat_netconn_sendq(struct netconn *conn)
 #if PF_PKT_SUPPORT
         case NETCONN_PKT_RAW:
             retVal = 0; /* always be 0 as frame send to driver directly */
-#endif
             break;
+#endif
         case NETCONN_UDP:
             retVal = netstat_udp_sendq(conn->pcb.udp);
             break;
@@ -3797,8 +3798,9 @@ u32_t osShellIpDebug(int argc, const char **argv)
             PRINTK("%-50s ", acIPv6Addr);
 
             if (snprintf_s(aclladdr, sizeof(aclladdr), sizeof(aclladdr) - 1, "%02X:%02X:%02X:%02X:%02X:%02X",
-                           neighbor_cache[i].lladdr[0], neighbor_cache[i].lladdr[1], neighbor_cache[i].lladdr[2],
-                           neighbor_cache[i].lladdr[3], neighbor_cache[i].lladdr[4], neighbor_cache[i].lladdr[5]) < 0) {
+                           neighbor_cache[i].lladdr[0], neighbor_cache[i].lladdr[1], /* 0, 1, member number */
+                           neighbor_cache[i].lladdr[2], neighbor_cache[i].lladdr[3], /* 2, 3, member number */
+                           neighbor_cache[i].lladdr[4], neighbor_cache[i].lladdr[5]) < 0) { /* 4, 5, member number */
                 return LOS_NOK;
             }
             PRINTK("%-25s ", aclladdr);
diff --git a/security/vid/vid.c b/security/vid/vid.c
index ba3f48f487e4bd36ecd885092145b042f1d0a853..0079fde1dca75ba53069f3a6684d4cb4332b8dc6 100644
--- a/security/vid/vid.c
+++ b/security/vid/vid.c
@@ -160,7 +160,7 @@ static VOID ReleaseVid(UINT16 vid)
     a = vid >> INT_BIT_SHIFT;
     b = vid & (INT_BIT_COUNT - 1);
 
-    idMap->bitMap[a] &= ~(1 << b);
+    idMap->bitMap[a] &= ~(1U << b);
 
     /* shrink bit map */
     if (mapMaxNum > 1) {
diff --git a/security/vid/vid_api.h b/security/vid/vid_api.h
index 12a7c75d7693d616e9977fbbcf3e9360e62367ce..ade2baac34631805618f5c9514694f8c0c0c62a4 100644
--- a/security/vid/vid_api.h
+++ b/security/vid/vid_api.h
@@ -34,9 +34,9 @@
 #include "los_typedef.h"
 #include "los_process_pri.h"
 
-#define INT_BIT_COUNT 32
-#define INT_BIT_SHIFT 5
-#define VID_MAP_MAX_NUM 32
+#define INT_BIT_COUNT 32U
+#define INT_BIT_SHIFT 5U
+#define VID_MAP_MAX_NUM 32U
 
 #define MAX_INVALID_TIMER_VID (VID_MAP_MAX_NUM * INT_BIT_COUNT)
 
diff --git a/shell/full/src/base/shell_lk.c b/shell/full/src/base/shell_lk.c
index 3320f1d9b7f8e7fa110c5904c2864ef096707c17..6716d71108ff604f1397cea018f6fe4bf47fdf1d 100644
--- a/shell/full/src/base/shell_lk.c
+++ b/shell/full/src/base/shell_lk.c
@@ -145,7 +145,7 @@ INT32 CmdLog(INT32 argc, const CHAR **argv)
         }
     } else if (!strncmp(argv[0], "module", strlen(argv[0]) + 1)) {
         module = strtoul(argv[1], &p, 0);
-        if ((*p != 0) || (module > MODULE4) || (module < MODULE0)) {
+        if ((*p != 0) || (module > MODULE4) || (module == MODULE0)) {
             PRINTK("log %s can't access %s\n", argv[0], argv[1]);
             PRINTK("not support yet\n");
             return -1;
diff --git a/shell/full/src/base/shmsg.c b/shell/full/src/base/shmsg.c
index cf07fa48eeab8b22a418e84b17c559fa216099ce..52f80ff05d64e490fdb02495e07902b2e285ab04 100644
--- a/shell/full/src/base/shmsg.c
+++ b/shell/full/src/base/shmsg.c
@@ -329,7 +329,7 @@ END:
 LITE_OS_SEC_TEXT_MINOR UINT32 ShellEntry(UINTPTR param)
 {
     CHAR ch;
-    INT32 n = 0;
+    INT32 n;
     ShellCB *shellCB = (ShellCB *)param;
 
     CONSOLE_CB *consoleCB = OsGetConsoleByID((INT32)shellCB->consoleID);
diff --git a/shell/full/src/cmds/dmesg.c b/shell/full/src/cmds/dmesg.c
index 619a65ab65797cdf515dbcfea3d4ef47e4010f20..a03abe81acb7306eadd20872c8597a124fb29098 100644
--- a/shell/full/src/cmds/dmesg.c
+++ b/shell/full/src/cmds/dmesg.c
@@ -425,7 +425,7 @@ STATIC VOID OsBufFullWrite(const CHAR *dst, UINT32 logLen)
 
 STATIC VOID OsWriteTailToHead(const CHAR *dst, UINT32 logLen)
 {
-    UINT32 writeLen = 0;
+    UINT32 writeLen;
     UINT32 bufSize = g_logBufSize;
     UINT32 logSize = g_dmesgInfo->logSize;
     UINT32 tail = g_dmesgInfo->logTail;
diff --git a/syscall/syscall_pub.c b/syscall/syscall_pub.c
index 32e8596cb6a143d312d08f584ec3d10e22355cb2..43e1e087e829e0524f390e8dd137d66304045c3d 100644
--- a/syscall/syscall_pub.c
+++ b/syscall/syscall_pub.c
@@ -83,6 +83,10 @@ int GetFullpath(int fd, const char *path, char **fullpath)
             fd = GetAssociatedSystemFd(fd);
         }
         ret = fs_getfilep(fd, &file);
+        if (ret < 0) {
+            ret = -EPERM;
+            goto OUT;
+        }
         if (file) {
             ret = stat(file->f_path, &bufRet);
             if (!ret) {
diff --git a/testsuites/kernel/sample/kernel_extend/cpup/smoke/It_extend_cpup_001.c b/testsuites/kernel/sample/kernel_extend/cpup/smoke/It_extend_cpup_001.c
index 41589d15c9673405b47489719a481b852bef936a..05f5866747df2b81a096c0da1a28df3b87e9d94a 100644
--- a/testsuites/kernel/sample/kernel_extend/cpup/smoke/It_extend_cpup_001.c
+++ b/testsuites/kernel/sample/kernel_extend/cpup/smoke/It_extend_cpup_001.c
@@ -39,8 +39,9 @@ extern "C" {
 
 static UINT32 TaskF02(VOID)
 {
-    UINT32 ret, cpupUse;
+    UINT32 ret = OS_ERROR, cpupUse;
     g_cpupTestCount++;
+
     // 2, Here, assert that g_cpupTestCount is equal to the expected value.
     ICUNIT_GOTO_EQUAL(g_cpupTestCount, 2, g_cpupTestCount, EXIT);
     cpupUse = LOS_HistoryProcessCpuUsage(g_testTaskID01, CPUP_ALL_TIME);
diff --git a/testsuites/kernel/sample/kernel_extend/cpup/smoke/It_extend_cpup_002.c b/testsuites/kernel/sample/kernel_extend/cpup/smoke/It_extend_cpup_002.c
index 1096be00b75e46222cfdfd7aa9b24b39f9b71c20..6e399ce32de445028465fe705db580dc0112e6b9 100644
--- a/testsuites/kernel/sample/kernel_extend/cpup/smoke/It_extend_cpup_002.c
+++ b/testsuites/kernel/sample/kernel_extend/cpup/smoke/It_extend_cpup_002.c
@@ -39,9 +39,10 @@ extern "C" {
 
 static UINT32 TaskF01(VOID)
 {
-    UINT32 ret;
+    UINT32 ret = OS_ERROR;
     UINT32 cpupUse;
     g_cpupTestCount++;
+
     ICUNIT_GOTO_EQUAL(g_cpupTestCount, 1, g_cpupTestCount, EXIT);
 
     cpupUse = LOS_HistoryProcessCpuUsage(LOS_GetCurrProcessID(), CPU_USE_MODE0);
diff --git a/testsuites/unittest/basic/exc/smoke/it_test_exc_001.cpp b/testsuites/unittest/basic/exc/smoke/it_test_exc_001.cpp
index b7330789a2562a0b850ec8bc79dda356e88d6977..26512427ba38e915e3278216edd064ffd07108df 100644
--- a/testsuites/unittest/basic/exc/smoke/it_test_exc_001.cpp
+++ b/testsuites/unittest/basic/exc/smoke/it_test_exc_001.cpp
@@ -33,12 +33,10 @@
 static int TestCase(void)
 {
     int ret;
-    int *test = NULL;
     int status = 0;
     pid_t pid = fork();
     ICUNIT_ASSERT_WITHIN_EQUAL(pid, 0, INVALID_PROCESS_ID, pid);
     if (pid == 0) {
-        *test = 0x1;
         exit(0);
     }
 
@@ -54,7 +52,6 @@ static int TestCase(void)
     pid = fork();
     ICUNIT_ASSERT_WITHIN_EQUAL(pid, 0, INVALID_PROCESS_ID, pid);
     if (pid == 0) {
-        *test = 0x1;
         exit(0);
     }
 
diff --git a/testsuites/unittest/basic/exc/smoke/it_test_exc_004.cpp b/testsuites/unittest/basic/exc/smoke/it_test_exc_004.cpp
index 6ad1462a57daab1f48b374c47fcb2d589bfaccb6..3299fefaccc9491aec84caed90d39e21240d2ab2 100644
--- a/testsuites/unittest/basic/exc/smoke/it_test_exc_004.cpp
+++ b/testsuites/unittest/basic/exc/smoke/it_test_exc_004.cpp
@@ -35,19 +35,15 @@
 
 static int TestThread(void)
 {
-    int *test = nullptr;
     int ret;
 
     pid_t pid = fork();
     ICUNIT_ASSERT_WITHIN_EQUAL(pid, 0, INVALID_PROCESS_ID, pid);
     if (pid == 0) {
-        *test = 0x1;
         while (1) {
         }
     }
 
-    *test = 0x1;
-
     ret = waitpid(pid, NULL, 0);
     ICUNIT_ASSERT_EQUAL(ret, pid, ret);
 
@@ -56,7 +52,6 @@ static int TestThread(void)
 
 static int TestCase(void)
 {
-    int *test = nullptr;
     int count = 5;
     int status = 0;
     int ret;
diff --git a/testsuites/unittest/extended/signal/smoke/mkfifo_test_001.cpp b/testsuites/unittest/extended/signal/smoke/mkfifo_test_001.cpp
index fb4c3d833860bc3d16be4587720afb911a229043..017bb106256cfb6a32f0cfa66dcb92f9778001dc 100644
--- a/testsuites/unittest/extended/signal/smoke/mkfifo_test_001.cpp
+++ b/testsuites/unittest/extended/signal/smoke/mkfifo_test_001.cpp
@@ -59,7 +59,7 @@ static int TestMkfifo()
         if (fd < 0) {
             return -1;
         }
-        write(fd, sentence, sizeof(sentence) + 1);
+        write(fd, sentence, sizeof(sentence));
         close(fd);
         exit(0);
     } else {
diff --git a/testsuites/unittest/extended/signal/smoke/mkfifo_test_002.cpp b/testsuites/unittest/extended/signal/smoke/mkfifo_test_002.cpp
index ffbf9fff8e4a92d1fbf98989124610ee7800a990..495a7330849921d397aa3bb32a3fb472df8feefd 100644
--- a/testsuites/unittest/extended/signal/smoke/mkfifo_test_002.cpp
+++ b/testsuites/unittest/extended/signal/smoke/mkfifo_test_002.cpp
@@ -45,7 +45,6 @@ static int TestMkfifoReturn()
     int retValue, i;
     int status = 0;
     pid_t pid;
-    char pathname[NAME_BUF_SIZE];
 
     retValue = mkfifo("/dev/fifo0", 0777); // 0777, mkfifo config.
     ICUNIT_ASSERT_EQUAL(retValue, 0, retValue);
@@ -57,7 +56,6 @@ static int TestMkfifoReturn()
     unlink("/dev/fifo0/fifo1");
     unlink("/dev/usr/fifo0");
     rmdir("/dev/usr");
-    unlink(pathname);
     unlink("/dev/fifo0");
     return LOS_OK;
 
@@ -66,7 +64,6 @@ EXIT:
     unlink("/dev/fifo0/fifo1");
     unlink("/dev/usr/fifo0");
     rmdir("/dev/usr");
-    unlink(pathname);
     unlink("/dev/fifo0");
     return LOS_NOK;
 }
diff --git a/testsuites/unittest/libc/io/full/IO_test_epoll_001.cpp b/testsuites/unittest/libc/io/full/IO_test_epoll_001.cpp
index e3e8fe8671e35fee71b9a8b55a15a0ee53f26ecd..9e5f45a5a04bb2bec48edfff17a25e74f848204c 100644
--- a/testsuites/unittest/libc/io/full/IO_test_epoll_001.cpp
+++ b/testsuites/unittest/libc/io/full/IO_test_epoll_001.cpp
@@ -54,7 +54,7 @@ static UINT32 testcase(VOID)
     UINT32 ret;
 
     retval = pipe(pipeFd);
-    ICUNIT_GOTO_EQUAL(retval, 0, retval, OUT);
+    ICUNIT_GOTO_EQUAL(retval, 0, retval, OUT4);
 
     /* Watch fd to see when it has input. */
     FD_ZERO(&rfds);
@@ -62,11 +62,11 @@ static UINT32 testcase(VOID)
 
     /* Wait up to three seconds. */
     epFd = epoll_create1(100); /* 100, cretae input, */
-    ICUNIT_GOTO_NOT_EQUAL(epFd, -1, epFd, OUT);
+    ICUNIT_GOTO_NOT_EQUAL(epFd, -1, epFd, OUT2);
 
     ev.events = EPOLLIN | EPOLLOUT | EPOLLRDNORM | EPOLLWRNORM;
     retval = epoll_ctl(epFd, EPOLL_CTL_ADD, pipeFd[0], &ev);
-    ICUNIT_GOTO_NOT_EQUAL(retval, -1, retval, OUT);
+    ICUNIT_GOTO_NOT_EQUAL(retval, -1, retval, OUT1);
 
     pid = fork();
     if (pid == 0) {
@@ -86,7 +86,7 @@ static UINT32 testcase(VOID)
         sleep(1);
         close(pipeFd[0]);
         retval = write(pipeFd[1], "0123456789012345678901234567890123456789", 40); /* write 40 bytes to stdin(fd 0) */
-        ICUNIT_GOTO_EQUAL(retval, 40, retval, OUT);
+        ICUNIT_GOTO_EQUAL(retval, 40, retval, OUT3);
         close(pipeFd[1]);
 
         wait(&status);
@@ -95,10 +95,20 @@ static UINT32 testcase(VOID)
     }
 
     return LOS_OK;
-OUT:
+OUT1:
+    close(epFd);
+    close(pipeFd[0]);
+    close(pipeFd[1]);
+    return LOS_NOK;
+OUT2:
     close(pipeFd[0]);
     close(pipeFd[1]);
+    return LOS_NOK;
+OUT3:
     close(epFd);
+    close(pipeFd[1]);
+    return LOS_NOK;
+OUT4:
     return LOS_NOK;
 }
 
diff --git a/testsuites/unittest/libc/io/full/IO_test_epoll_002.cpp b/testsuites/unittest/libc/io/full/IO_test_epoll_002.cpp
index 1c59173e898a4a9db802124b15885fdacbd4de0f..1d46e936f96e62ccbf87f18f956ee5f966cdad60 100644
--- a/testsuites/unittest/libc/io/full/IO_test_epoll_002.cpp
+++ b/testsuites/unittest/libc/io/full/IO_test_epoll_002.cpp
@@ -53,7 +53,7 @@ static UINT32 testcase(VOID)
     int i = 0;
     int status;
 
-    int epFd;
+    int epFd = 0;
     sigset_t mask;
     void (*retSig)(int);
     struct epoll_event ev;
@@ -75,7 +75,7 @@ static UINT32 testcase(VOID)
     ICUNIT_ASSERT_EQUAL(retval, 0, retval);
 
     retval = pipe(pipeFd);
-    ICUNIT_GOTO_EQUAL(retval, 0, retval, OUT);
+    ICUNIT_GOTO_EQUAL(retval, 0, retval, OUT3);
 
     /* Watch fd to see when it has input. */
     FD_ZERO(&rfds);
@@ -86,12 +86,12 @@ static UINT32 testcase(VOID)
     tv.tv_nsec = 5; /* 5, wait timer, nano second */
 
     epFd = epoll_create1(100); /* 100, cretae input, */
-    ICUNIT_GOTO_NOT_EQUAL(epFd, -1, epFd, OUT);
+    ICUNIT_GOTO_NOT_EQUAL(epFd, -1, epFd, OUT2);
 
     ev.events = EPOLLRDNORM;
     retval = epoll_ctl(epFd, EPOLL_CTL_ADD, pipeFd[0], &ev);
 
-    ICUNIT_GOTO_NOT_EQUAL(retval, -1, retval, OUT);
+    ICUNIT_GOTO_NOT_EQUAL(retval, -1, retval, OUT1);
 
     pid = fork();
     if (pid == 0) {
@@ -125,10 +125,12 @@ static UINT32 testcase(VOID)
     }
 
     return LOS_OK;
-OUT:
+OUT1:
+    close(epFd);
+OUT2:
     close(pipeFd[0]);
     close(pipeFd[1]);
-    close(epFd);
+OUT3:
     return LOS_NOK;
 }
 
diff --git a/testsuites/unittest/process/basic/process/smoke/process_test_002.cpp b/testsuites/unittest/process/basic/process/smoke/process_test_002.cpp
index 59a157eded8c8bdeaed7e9a59ff02caa30d3258e..5badaa3aecaa160a90ad36bef5483b4ed0bcbbf2 100644
--- a/testsuites/unittest/process/basic/process/smoke/process_test_002.cpp
+++ b/testsuites/unittest/process/basic/process/smoke/process_test_002.cpp
@@ -36,7 +36,7 @@ static int Testcase()
 #define CURRENT_PROCESS_POLICY SCHED_RR
 
     struct sched_param param = { 0 };
-    int ret;
+    int ret = OS_ERROR;
 
     int currProcessPri = getpriority(PRIO_PROCESS, getpid());
     ICUNIT_GOTO_WITHIN_EQUAL(currProcessPri, 0, 31, currProcessPri, ERROR_OUT); // 31, assert function Result equal to this.
diff --git a/testsuites/unittest/process/basic/process/smoke/process_test_013.cpp b/testsuites/unittest/process/basic/process/smoke/process_test_013.cpp
index 76a58d538d539e9da07bbfd5b682a645fb14197d..613d09123b4ea6b99c8d5f7be44b63d7fe9350a0 100644
--- a/testsuites/unittest/process/basic/process/smoke/process_test_013.cpp
+++ b/testsuites/unittest/process/basic/process/smoke/process_test_013.cpp
@@ -35,7 +35,7 @@ static int TestCase()
 {
 #define PROCESS_TEST_PRI1 (currProcessPri + 1)
     struct sched_param param = { 0 };
-    int ret;
+    int ret = OS_ERROR;
     int val, currPolicy;
     int currProcessPri = getpriority(PRIO_PROCESS, getpid());
     ICUNIT_GOTO_WITHIN_EQUAL(currProcessPri, 0, 31, currProcessPri, ERROR_OUT); // 31, assert currProcessPri equal to this.