diff --git a/services/modules/seccomp/seccomp_policy.c b/services/modules/seccomp/seccomp_policy.c index da3431ed9b073c878c21b843a1ecfc5803bb84c7..b5828dcd12a0cfb799f0c70621030a6845bb5033 100644 --- a/services/modules/seccomp/seccomp_policy.c +++ b/services/modules/seccomp/seccomp_policy.c @@ -33,7 +33,7 @@ static bool IsSupportFilterFlag(unsigned int filterFlag) { errno = 0; - int ret = syscall(__NR_seccomp, SECCOMP_SET_MODE_FILTER, filterFlag, NULL); + long ret = syscall(__NR_seccomp, SECCOMP_SET_MODE_FILTER, filterFlag, NULL); if (ret != -1 || errno != EFAULT) { SECCOMP_LOGE("not support seccomp flag %u", filterFlag); return false; diff --git a/services/param/adapter/param_selinux.c b/services/param/adapter/param_selinux.c index 1d50a13437a905cf1e991c0f24814720974e414e..7c276b119861956b786d535a418c6ab6ce5e988d 100644 --- a/services/param/adapter/param_selinux.c +++ b/services/param/adapter/param_selinux.c @@ -92,7 +92,7 @@ static int InitLocalSecurityLabel(ParamSecurityLabel *security, int isInit) security->flags[PARAM_SECURITY_SELINUX] = 0; PARAM_LOGV("InitLocalSecurityLabel"); #if !(defined STARTUP_INIT_TEST || defined LOCAL_TEST) - if (isInit) { + if ((bool)isInit) { int ret = InitSelinuxOpsForInit(&GetParamWorkSpace()->selinuxSpace); PARAM_CHECK(ret == 0, return -1, "Failed to init selinux ops"); } else { diff --git a/services/param/base/param_base.c b/services/param/base/param_base.c index 6d7c9ebdf1dfaf2c8a43cbfe0fb1abc80b46b5a1..a37f5fd6e09caaa19c4a7cb92845f4146ece9fbd 100644 --- a/services/param/base/param_base.c +++ b/services/param/base/param_base.c @@ -105,7 +105,7 @@ static int CheckNeedInit(int onlyRead, const PARAM_WORKSPACE_OPS *ops) g_paramWorkSpace.ops.setfilecon = ops->setfilecon; #endif } - PARAM_LOGI("InitParamWorkSpace %p %x", &g_paramWorkSpace, g_paramWorkSpace.flags); + PARAM_LOGI("InitParamWorkSpace %x", g_paramWorkSpace.flags); if (PARAM_TEST_FLAG(g_paramWorkSpace.flags, WORKSPACE_FLAGS_INIT)) { return 0; } @@ -118,7 +118,7 @@ static int CheckNeedInit(int onlyRead, const PARAM_WORKSPACE_OPS *ops) } // for ut, do not init workspace char path[PATH_MAX] = { 0 }; - (void)readlink("/proc/self/exe", path, sizeof(path)); + (void)readlink("/proc/self/exe", path, sizeof(path) - 1); char *name = strrchr(path, '/'); if (name != NULL) { name++;