Fix crash and sync

Author: yujincheng08

manager/src/main/java/org/lsposed/lspatch/manager/ManagerService.kt

@@ -15,19 +15,19 @@ object ManagerService : ILSPApplicationService.Stub() {
 
     private const val TAG = "ManagerService"
 
-    override fun getLegacyModulesList(): MutableList<Module> {
-        TODO("Not yet implemented")
-    }
-
-    override fun getModulesList(): List<Module> {
+    override fun getLegacyModulesList(): List<Module> {
         val app = lspApp.packageManager.getNameForUid(Binder.getCallingUid())
         val list = app?.let {
             runBlocking { ConfigManager.getModuleFilesForApp(it) }
         }.orEmpty()
-        Log.d(TAG, "$app calls getModulesList: $list")
+        Log.d(TAG, "$app calls getLegacyModulesList: $list")
         return list
     }
 
+    override fun getModulesList(): List<Module> {
+        return emptyList()
+    }
+
     override fun getPrefsPath(packageName: String): String {
         TODO("Not yet implemented")
     }

patch-loader/src/main/java/org/lsposed/lspatch/loader/LSPApplication.java

@@ -85,8 +85,7 @@ public static void onLoad() throws RemoteException, IOException {
         }
 
         disableProfile(context);
-        Startup.initXposed(false, ActivityThread.currentProcessName(), ActivityThread.currentApplication().getPackageCodePath(), service);
-        Log.i(TAG, "Bootstrap Xposed");
+        Startup.initXposed(false, ActivityThread.currentProcessName(), context.getApplicationInfo().dataDir, service);
         Startup.bootstrapXposed();
         // WARN: Since it uses `XResource`, the following class should not be initialized
         // before forkPostCommon is invoke. Otherwise, you will get failure of XResources

patch-loader/src/main/java/org/lsposed/lspatch/service/LocalApplicationService.java

@@ -58,13 +58,13 @@ public LocalApplicationService(Context context) {
     }
 
     @Override
-    public List<Module> getLegacyModulesList() throws RemoteException {
-        return null;
+    public List<Module> getLegacyModulesList() {
+        return modules;
     }
 
     @Override
     public List<Module> getModulesList() {
-        return modules;
+        return new ArrayList<>();
     }
 
     @Override
@@ -76,4 +76,9 @@ public String getPrefsPath(String packageName) {
     public ParcelFileDescriptor requestInjectedManagerBinder(List<IBinder> binder) {
         return null;
     }
+
+    @Override
+    public IBinder asBinder() {
+        return this;
+    }
 }

patch-loader/src/main/java/org/lsposed/lspatch/service/RemoteApplicationService.java

@@ -34,7 +34,7 @@ public class RemoteApplicationService implements ILSPApplicationService {
     private static final String TAG = "LSPatch";
     private static final String MODULE_SERVICE = Constants.MANAGER_PACKAGE_NAME + ".manager.ModuleService";
 
-    private ILSPApplicationService service;
+    private volatile ILSPApplicationService service;
 
     @SuppressLint("DiscouragedPrivateApi")
     public RemoteApplicationService(Context context) throws RemoteException {
@@ -48,8 +48,8 @@ public RemoteApplicationService(Context context) throws RemoteException {
                 @Override
                 public void onServiceConnected(ComponentName name, IBinder binder) {
                     Log.i(TAG, "Manager binder received");
-                    latch.countDown();
                     service = Stub.asInterface(binder);
+                    latch.countDown();
                 }
 
                 @Override