diff options
| author | 2023-03-13 17:39:27 +0000 | |
|---|---|---|
| committer | 2023-03-13 17:39:27 +0000 | |
| commit | 087c3f92270173a61e10d384dafb1cda995d2aef (patch) | |
| tree | 47148c3e39de60f0621d841208c7d74a952ea1a7 | |
| parent | 942bc62c4f3878a17cb1e921a301a7eed47a5962 (diff) | |
| parent | 97735895a286f82ce9d955e043f3d28810e9e268 (diff) | |
Merge "Treat process group creation failure due to a dead process as non-fatal" am: 03dfa27d40 am: 97735895a2
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/2467472
Change-Id: I31ad8df3a5ff416bc1e739dfb9d9d00ab636d691
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| -rw-r--r-- | core/jni/com_android_internal_os_Zygote.cpp | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/core/jni/com_android_internal_os_Zygote.cpp b/core/jni/com_android_internal_os_Zygote.cpp index f04e2457d51f..57a196fa3d0c 100644 --- a/core/jni/com_android_internal_os_Zygote.cpp +++ b/core/jni/com_android_internal_os_Zygote.cpp @@ -1805,10 +1805,15 @@ static void SpecializeCommon(JNIEnv* env, uid_t uid, gid_t gid, jintArray gids, if (!is_system_server && getuid() == 0) { const int rc = createProcessGroup(uid, getpid()); if (rc != 0) { - fail_fn(rc == -EROFS ? CREATE_ERROR("createProcessGroup failed, kernel missing " - "CONFIG_CGROUP_CPUACCT?") - : CREATE_ERROR("createProcessGroup(%d, %d) failed: %s", uid, - /* pid= */ 0, strerror(-rc))); + if (rc == -ESRCH) { + // If process is dead, treat this as a non-fatal error + ALOGE("createProcessGroup(%d, %d) failed: %s", uid, /* pid= */ 0, strerror(-rc)); + } else { + fail_fn(rc == -EROFS ? CREATE_ERROR("createProcessGroup failed, kernel missing " + "CONFIG_CGROUP_CPUACCT?") + : CREATE_ERROR("createProcessGroup(%d, %d) failed: %s", uid, + /* pid= */ 0, strerror(-rc))); + } } } |