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

Commit 03637a45 authored by Colin Cross's avatar Colin Cross Committed by Automerger Merge Worker
Browse files

Merge "Enable ptrace on userdebug when JDWP is disabled" into main am:... 

Merge "Enable ptrace on userdebug when JDWP is disabled" into main am: e59b7500 am: ef0fcdb1 am: d75cd149 am: 42f2b59a am: 637d7581

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



Change-Id: I6b600738b836291b1358df87d1aae2519d723f85
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents eaa15ab1 637d7581

core/java/com/android/internal/os/Zygote.java

+8 −0
Original line number Diff line number Diff line
@@ -195,6 +195,11 @@ public final class Zygote { 
     */

    public static final int PROFILEABLE = 1 << 24;



    /**

     * Enable ptrace.  This is enabled on eng or userdebug builds, or if the app is debuggable.

     */

    public static final int DEBUG_ENABLE_PTRACE = 1 << 25;



    /** No external storage should be mounted. */

    public static final int MOUNT_EXTERNAL_NONE = IVold.REMOUNT_MODE_NONE;

    /** Default external storage should be mounted. */
@@ -1028,6 +1033,9 @@ public final class Zygote { 
        if (Build.IS_ENG || ENABLE_JDWP) {

            args.mRuntimeFlags |= Zygote.DEBUG_ENABLE_JDWP;

        }

        if (RoSystemProperties.DEBUGGABLE) {

            args.mRuntimeFlags |= Zygote.DEBUG_ENABLE_PTRACE;

        }

    }



    /**

core/jni/com_android_internal_os_Zygote.cpp

+4 −1
Original line number Diff line number Diff line
@@ -356,6 +356,7 @@ enum RuntimeFlags : uint32_t { 
    GWP_ASAN_LEVEL_DEFAULT = 3 << 21,

    NATIVE_HEAP_ZERO_INIT_ENABLED = 1 << 23,

    PROFILEABLE = 1 << 24,

    DEBUG_ENABLE_PTRACE = 1 << 25,

};



enum UnsolicitedZygoteMessageTypes : uint32_t {
@@ -1887,8 +1888,10 @@ static void SpecializeCommon(JNIEnv* env, uid_t uid, gid_t gid, jintArray gids, 
    }



    // Set process properties to enable debugging if required.

    if ((runtime_flags & RuntimeFlags::DEBUG_ENABLE_JDWP) != 0) {

    if ((runtime_flags & RuntimeFlags::DEBUG_ENABLE_PTRACE) != 0) {

        EnableDebugger();

        // Don't pass unknown flag to the ART runtime.

        runtime_flags &= ~RuntimeFlags::DEBUG_ENABLE_PTRACE;

    }

    if ((runtime_flags & RuntimeFlags::PROFILE_FROM_SHELL) != 0) {

        // simpleperf needs the process to be dumpable to profile it.

services/core/java/com/android/server/am/ProcessList.java

+1 −0
Original line number Diff line number Diff line
@@ -1827,6 +1827,7 @@ public final class ProcessList { 


            if (debuggableFlag) {

                runtimeFlags |= Zygote.DEBUG_ENABLE_JDWP;

                runtimeFlags |= Zygote.DEBUG_ENABLE_PTRACE;

                runtimeFlags |= Zygote.DEBUG_JAVA_DEBUGGABLE;

                // Also turn on CheckJNI for debuggable apps. It's quite

                // awkward to turn on otherwise.