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

Commit 637d7581 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

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



Change-Id: I5951217401f4b1f678cc4ded8c53fbbb0f958b5a
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 979974f1 42f2b59a

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.