diff --git a/appdata-sandbox-asan.json b/appdata-sandbox-asan.json index 06c592a405bef40dc9a665c5262c1b08565ec9b3..0fb704fae3ae60155d20d773d0dcf7375035a834 100755 --- a/appdata-sandbox-asan.json +++ b/appdata-sandbox-asan.json @@ -2,7 +2,7 @@ "common" : [{ "top-sandbox-switch": "ON", "app-base" : [{ - "sandbox-root" : "/mnt/sandbox/", + "sandbox-root" : "/mnt/sandbox//", "sandbox-ns-flags" : [], "mount-paths" : [ { "src-path" : "/system/asan/lib", diff --git a/appdata-sandbox64.json b/appdata-sandbox64.json index 1167b4646cf8909c31d0030622c80bb7b82c134b..2633f72f212d0aae1d777981b0d93177298f6446 100644 --- a/appdata-sandbox64.json +++ b/appdata-sandbox64.json @@ -2,7 +2,7 @@ "common" : [{ "top-sandbox-switch": "ON", "app-base" : [{ - "sandbox-root" : "/mnt/sandbox/", + "sandbox-root" : "/mnt/sandbox//", "sandbox-ns-flags" : [ "net" ], "mount-paths" : [{ "src-path" : "/system/lib64", @@ -39,7 +39,7 @@ ] }], "app-resources" : [{ - "sandbox-root" : "/mnt/sandbox/", + "sandbox-root" : "/mnt/sandbox//", "mount-paths" : [], "flags-point" : [], "symbol-links" : [] diff --git a/modules/sandbox/appspawn_sandbox.c b/modules/sandbox/appspawn_sandbox.c index 1b5dfd02336a86838c79262a8ac04e133598d710..7f98cb61c2fdba265148c2b70b3cb86bd09a0e72 100644 --- a/modules/sandbox/appspawn_sandbox.c +++ b/modules/sandbox/appspawn_sandbox.c @@ -227,7 +227,6 @@ static int InitSandboxContext(SandboxContext *context, } context->message = property->message; context->sandboxNsFlags = CLONE_NEWNS; - if (NeedNetworkIsolated(context, property)) { context->sandboxNsFlags |= sandbox->sandboxNsFlags & CLONE_NEWNET ? CLONE_NEWNET : 0; } diff --git a/modules/sandbox/sandbox_utils.cpp b/modules/sandbox/sandbox_utils.cpp index 1a6bb911d0b14bf3d39e91f09661c1e78803ee73..605eceb950de80fa6ae161e7bdda0c076b98b437 100644 --- a/modules/sandbox/sandbox_utils.cpp +++ b/modules/sandbox/sandbox_utils.cpp @@ -322,7 +322,7 @@ int32_t SandboxUtils::DoAppSandboxMountOnce(const char *originPath, const char * if (originPath == nullptr || destinationPath == nullptr || originPath[0] == '\0' || destinationPath[0] == '\0') { return 0; } - if (originPath != nullptr && strstr(originPath, "system/etc/hosts") != nullptr) { + if (strstr(originPath, "system/etc/hosts") != nullptr) { CheckAndCreatFile(destinationPath); } else { MakeDirRecursive(destinationPath, FILE_MODE); @@ -333,7 +333,7 @@ int32_t SandboxUtils::DoAppSandboxMountOnce(const char *originPath, const char * struct timespec mountStart = {0}; clock_gettime(CLOCK_MONOTONIC_COARSE, &mountStart); APPSPAWN_LOGV("Bind mount %{public}s to %{public}s '%{public}s' '%{public}lu' '%{public}s' '%{public}u'", - originPath, destinationPath, fsType, mountFlags, options, mountSharedFlag); + originPath, destinationPath, fsType, mountFlags, options, mountSharedFlag); ret = mount(originPath, destinationPath, fsType, mountFlags, options); struct timespec mountEnd = {0}; clock_gettime(CLOCK_MONOTONIC_COARSE, &mountEnd); diff --git a/standard/BUILD.gn b/standard/BUILD.gn index 06e355397d09c663544c8d3e04517d02872196d8..ae540d4da6440da2cb4ed3ed384f53e517813d10 100644 --- a/standard/BUILD.gn +++ b/standard/BUILD.gn @@ -195,14 +195,7 @@ ohos_executable("cjappspawn") { "hitrace:hitrace_meter", "init:libbegetutil", ] - if (appspawn_report_event) { - defines += [ "APPSPAWN_HISYSEVENT" ] - external_deps += [ "hisysevent:libhisysevent" ] - sources += [ - "${appspawn_path}/modules/sysevent/appspawn_hisysevent.cpp", - "${appspawn_path}/modules/sysevent/hisysevent_adapter.cpp", - ] - } + if (build_selinux) { defines += [ "WITH_SELINUX" ] external_deps += [ @@ -270,14 +263,7 @@ ohos_executable("nativespawn") { "hitrace:hitrace_meter", "init:libbegetutil", ] - if (appspawn_report_event) { - defines += [ "APPSPAWN_HISYSEVENT" ] - external_deps += [ "hisysevent:libhisysevent" ] - sources += [ - "${appspawn_path}/modules/sysevent/appspawn_hisysevent.cpp", - "${appspawn_path}/modules/sysevent/hisysevent_adapter.cpp", - ] - } + if (build_selinux) { defines += [ "WITH_SELINUX" ] external_deps += [ diff --git a/standard/appspawn_service.c b/standard/appspawn_service.c index 3401316ce72d007bb1c39d4ef594d36b7e5d2641..aae3dd26d27a92711ee1102148f3deba9f44d076 100644 --- a/standard/appspawn_service.c +++ b/standard/appspawn_service.c @@ -1084,7 +1084,6 @@ static void ProcessChildResponse(const WatcherHandle taskHandle, int fd, uint32_ } #endif clock_gettime(CLOCK_MONOTONIC, &appInfo->spawnEnd); - // add max info #ifdef APPSPAWN_HISYSEVENT //add process spawn duration into hisysevent,(ms) @@ -1671,7 +1670,6 @@ APPSPAWN_STATIC int AppspawnDevicedebugDeal(const char* op, int pid, cJSON *args if (strcmp(op, "kill") == 0) { return AppspawpnDevicedebugKill(pid, args); } - APPSPAWN_LOGE("appspawn devicedebug op:%{public}s invaild", op); return -1; diff --git a/test/unittest/app_spawn_standard_test/app_spawn_service_test.cpp b/test/unittest/app_spawn_standard_test/app_spawn_service_test.cpp index 18b4a860f3378b946405df1864e5a164ffee496a..84e41d9d3000dd23d69a4905b058034268e13fbe 100644 --- a/test/unittest/app_spawn_standard_test/app_spawn_service_test.cpp +++ b/test/unittest/app_spawn_standard_test/app_spawn_service_test.cpp @@ -790,7 +790,7 @@ HWTEST_F(AppSpawnServiceTest, App_Spawn_InitCommonEnv_001, TestSize.Level0) env = getenv("PATH"); EXPECT_NE(env, nullptr); if (env != nullptr) { - EXPECT_NE((uint64_t)strstr(env, "/data/app/bin:/data/service/hnp/bin"), 0); + EXPECT_NE(strstr(env, "/data/app/bin:/data/service/hnp/bin"), nullptr); } } env = getenv("HOME");