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

Revert "Remove exception from OverlayConfig zygote method" am: 920105c38b am: b145349241

Browse Source 
Change-Id: I2c226a24994b77201e983dc4f45c111bdb92cf7b
This commit is contained in:
Automerger Merge Worker
2020-03-03 10:27:47 +00:00
parent e135ba6170 b145349241
commit 12f3b82bc9
1 changed files with 18 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
core/java/com/android/internal/content/om/OverlayConfig.java
View File

@@ -21,6 +21,7 @@ import android.annotation.Nullable;
import android.content.pm.PackagePartitions; import android.content.pm.PackagePartitions;
import android.content.pm.parsing.ParsingPackageRead; import android.content.pm.parsing.ParsingPackageRead;
import android.os.Build; import android.os.Build;
import android.os.Process;
import android.os.Trace; import android.os.Trace;
import android.util.ArrayMap; import android.util.ArrayMap;
import android.util.Log; import android.util.Log;
@@ -185,6 +186,13 @@ public class OverlayConfig {
*/ */
@NonNull @NonNull
public static OverlayConfig getZygoteInstance() { public static OverlayConfig getZygoteInstance() {
if (Process.myUid() != Process.ROOT_UID) {
// Scan the overlays in the zygote process to generate configuration settings for
// overlays on the system image. Do not cache this instance so OverlayConfig will not
// be present in applications by default.
throw new IllegalStateException("Can only be invoked in the root process");
}
Trace.traceBegin(Trace.TRACE_TAG_RRO, "OverlayConfig#getZygoteInstance"); Trace.traceBegin(Trace.TRACE_TAG_RRO, "OverlayConfig#getZygoteInstance");
try { try {
return new OverlayConfig(null /* rootDirectory */, OverlayScanner::new, return new OverlayConfig(null /* rootDirectory */, OverlayScanner::new,
@@ -201,12 +209,13 @@ public class OverlayConfig {
*/ */
@NonNull @NonNull
public static OverlayConfig initializeSystemInstance(PackageProvider packageProvider) { public static OverlayConfig initializeSystemInstance(PackageProvider packageProvider) {
Trace.traceBegin(Trace.TRACE_TAG_RRO, "OverlayConfig#initializeSystemInstance"); if (Process.myUid() != Process.SYSTEM_UID) {
try { throw new IllegalStateException("Can only be invoked in the system process");
sInstance = new OverlayConfig(null, null, packageProvider);
} finally {
Trace.traceEnd(Trace.TRACE_TAG_RRO);
} }
Trace.traceBegin(Trace.TRACE_TAG_RRO, "OverlayConfig#initializeSystemInstance");
sInstance = new OverlayConfig(null, null, packageProvider);
Trace.traceEnd(Trace.TRACE_TAG_RRO);
return sInstance; return sInstance;
} }
@@ -370,6 +379,10 @@ public class OverlayConfig {
*/ */
@NonNull @NonNull
public String[] createImmutableFrameworkIdmapsInZygote() { public String[] createImmutableFrameworkIdmapsInZygote() {
if (Process.myUid() != Process.ROOT_UID) {
throw new IllegalStateException("This method can only be called from the root process");
}
final String targetPath = "/system/framework/framework-res.apk"; final String targetPath = "/system/framework/framework-res.apk";
final ArrayList<String> idmapPaths = new ArrayList<>(); final ArrayList<String> idmapPaths = new ArrayList<>();
final ArrayList<IdmapInvocation> idmapInvocations = final ArrayList<IdmapInvocation> idmapInvocations =