Bug 1520: Android 14 (SDK API 34): Fix Writable dex file is not allowed

sgothel · sgothel · commit 1afa0e423e5c · 2025-07-14T02:54:44.000+02:00
Simply set the copied apk dex-file to read-only.
diff --git a/src/java/jogamp/android/launcher/ClassLoaderUtil.java b/src/java/jogamp/android/launcher/ClassLoaderUtil.java
@@ -168,11 +168,12 @@ private static synchronized ClassLoader createClassLoaderImpl(final Context ctx,
                        final File dst = new File(userAPK);
                        try {
                            copyFile(src, dst);
+                           dst.setReadOnly();
                        } catch (final IOException e) {
                            Log.d(TAG, "error copying <"+src+"> -> <"+dst+">: "+e, e);
                            return null;
                        }
-                       Log.d(TAG, "APK["+apkCount+"] copied: <"+src+"> -> <"+dst+">");
+                       Log.d(TAG, "APK["+apkCount+"] copied: <"+src+"> -> <"+dst+">, writable="+dst.canWrite());
                    }
                    apks.append(userAPK);
                    Log.d(TAG, "APK["+apkCount+"] found: <"+lastUserPackageName+"> -> <"+userAPK+">");

Original file line number	Diff line number	Diff line change
`@@ -168,11 +168,12 @@ private static synchronized ClassLoader createClassLoaderImpl(final Context ctx,`
`168`	`168`	`final File dst = new File(userAPK);`
`169`	`169`	`try {`
`170`	`170`	`copyFile(src, dst);`
	`171`	`+ dst.setReadOnly();`
`171`	`172`	`} catch (final IOException e) {`
`172`	`173`	`Log.d(TAG, "error copying <"+src+"> -> <"+dst+">: "+e, e);`
`173`	`174`	`return null;`
`174`	`175`	`}`
`175`		`- Log.d(TAG, "APK["+apkCount+"] copied: <"+src+"> -> <"+dst+">");`
	`176`	`+ Log.d(TAG, "APK["+apkCount+"] copied: <"+src+"> -> <"+dst+">, writable="+dst.canWrite());`
`176`	`177`	`}`
`177`	`178`	`apks.append(userAPK);`
`178`	`179`	`Log.d(TAG, "APK["+apkCount+"] found: <"+lastUserPackageName+"> -> <"+userAPK+">");`