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

Commit 5cba0cff authored by Kiyoung Kim's avatar Kiyoung Kim
Browse files

Add product apk support from libnativeloader 

Currently libnativeloader appends vendor lib path if apk is from vendor
partition. Similar to this we should add product lib path if apk if from product partition.

Bug: 124705551
Test: m -j && tested from crosshatch with product apps (such as Wallpaper)
Change-Id: Ic85b16e8e4484a35c659c16e067f1e7c7285d05e
Merged-In: Iaa8d0ea61b1b9ab8a1da676a492f75c6a5d71d5d
parent 06d80679

config/hiddenapi-greylist.txt

+1 −1
Original line number Diff line number Diff line
@@ -1419,7 +1419,7 @@ Lcom/android/internal/os/BaseCommand;->mArgs:Landroid/os/ShellCommand; 
Lcom/android/internal/os/BatterySipper$DrainType;->values()[Lcom/android/internal/os/BatterySipper$DrainType;

Lcom/android/internal/os/BinderInternal;->getContextObject()Landroid/os/IBinder;

Lcom/android/internal/os/BinderInternal;->handleGc()V

Lcom/android/internal/os/ClassLoaderFactory;->createClassloaderNamespace(Ljava/lang/ClassLoader;ILjava/lang/String;Ljava/lang/String;ZZ)Ljava/lang/String;

Lcom/android/internal/os/ClassLoaderFactory;->createClassloaderNamespace(Ljava/lang/ClassLoader;ILjava/lang/String;Ljava/lang/String;ZLjava/lang/String;)Ljava/lang/String;

Lcom/android/internal/os/IDropBoxManagerService$Stub;->asInterface(Landroid/os/IBinder;)Lcom/android/internal/os/IDropBoxManagerService;

Lcom/android/internal/os/ProcessCpuTracker$Stats;->name:Ljava/lang/String;

Lcom/android/internal/os/ProcessCpuTracker$Stats;->rel_stime:I

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

+2 −9
Original line number Diff line number Diff line
@@ -115,20 +115,13 @@ public class ClassLoaderFactory { 
        final ClassLoader classLoader = createClassLoader(dexPath, librarySearchPath, parent,

                classLoaderName, sharedLibraries);



        boolean isForVendor = false;

        for (String path : dexPath.split(":")) {

            if (path.startsWith("/vendor/")) {

                isForVendor = true;

                break;

            }

        }

        Trace.traceBegin(Trace.TRACE_TAG_ACTIVITY_MANAGER, "createClassloaderNamespace");

        String errorMessage = createClassloaderNamespace(classLoader,

                                                         targetSdkVersion,

                                                         librarySearchPath,

                                                         libraryPermittedPath,

                                                         isNamespaceShared,

                                                         isForVendor);

                                                         dexPath);

        Trace.traceEnd(Trace.TRACE_TAG_ACTIVITY_MANAGER);



        if (errorMessage != null) {
@@ -144,5 +137,5 @@ public class ClassLoaderFactory { 
                                                            String librarySearchPath,

                                                            String libraryPermittedPath,

                                                            boolean isNamespaceShared,

                                                            boolean isForVendor);

                                                            String dexPath);

}

core/jni/com_android_internal_os_ClassLoaderFactory.cpp

+3 −3
Original line number Diff line number Diff line
@@ -28,16 +28,16 @@ static jstring createClassloaderNamespace_native(JNIEnv* env, 
                                              jstring librarySearchPath,

                                              jstring libraryPermittedPath,

                                              jboolean isShared,

                                              jboolean isForVendor) {

                                              jstring dexPath) {

    return android::CreateClassLoaderNamespace(env, targetSdkVersion,

                                               classLoader, isShared == JNI_TRUE,

                                               isForVendor == JNI_TRUE,

                                               dexPath,

                                               librarySearchPath, libraryPermittedPath);

}



static const JNINativeMethod g_methods[] = {

    { "createClassloaderNamespace",

      "(Ljava/lang/ClassLoader;ILjava/lang/String;Ljava/lang/String;ZZ)Ljava/lang/String;",

      "(Ljava/lang/ClassLoader;ILjava/lang/String;Ljava/lang/String;ZLjava/lang/String;)Ljava/lang/String;",

      reinterpret_cast<void*>(createClassloaderNamespace_native) },

};