Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Enable ptrace on userdebug when JDWP is disabled

Browse Source 
r.android.com/2217921 disabled JDWP on userdebug builds by default in
order to avoid the JDWP overhead causing differences between user and
userdebug builds.  This had the side effect of disabling ptrace, which
means libmemunreachable can no longer ptrace app processes, and app
processes are no longer debuggable by gdbserver unless it is run as
root.

Add a new DEBUG_ENABLE_PTRACE zygote runtimeFlag for enabling ptrace,
and set it anywhere DEBUG_ENABLE_JDWP was set before
r.android.com/2217921.  Use the flag to call EnableDebugging.

Bug: 285967712
Test: adb shell dumpsys meminfo --unreachable com.android.gallery3d
Change-Id: I2fb1aa0aea81400f573b8c2e2fa309b7942d9446
This commit is contained in:
Colin Cross
2023-07-06 16:17:01 -07:00
parent eae2d15e64
commit 990b5ec721
3 changed files with 13 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
core/java/com/android/internal/os/Zygote.java
View File

@@ -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;
}
}
/**

5
core/jni/com_android_internal_os_Zygote.cpp
View File

@@ -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.

1
services/core/java/com/android/server/am/ProcessList.java
View File

@@ -1741,6 +1741,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.