Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 37075911 authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Treat process group creation failure due to a dead process as... 

Merge "Treat process group creation failure due to a dead process as non-fatal" am: 03dfa27d am: 97735895 am: 087c3f92 am: 931bdd37

Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/2467472



Change-Id: Icd7fe6448c9e9748a681708ca6c775493a906339
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents ff1c12b1 931bdd37

core/jni/com_android_internal_os_Zygote.cpp

+9 −4
Original line number Diff line number Diff line
@@ -1805,12 +1805,17 @@ 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) {

            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)));

            }

        }

    }



    SetGids(env, gids, is_child_zygote, fail_fn);

    SetRLimits(env, rlimits, fail_fn);