fixes: termux#3565 - remove canRead check for scoped storage compat

billybox1926-jpg · billybox1926-jpg · commit 5d50a85feb62 · 2026-06-11T20:29:17.000-07:00
diff --git a/app/src/main/java/com/termux/app/TermuxOpenReceiver.java b/app/src/main/java/com/termux/app/TermuxOpenReceiver.java
@@ -81,10 +81,12 @@ public void onReceive(Context context, Intent intent) {
         }
 
         final File fileToShare = new File(filePath);
-        if (!(fileToShare.isFile() && fileToShare.canRead())) {
-            Logger.logError(LOG_TAG, "Not a readable file: '" + fileToShare.getAbsolutePath() + "'");
+        if (!fileToShare.isFile()) {
+            Logger.logError(LOG_TAG, "Not a file: '" + fileToShare.getAbsolutePath() + "'");
             return;
         }
+        // Note: canRead() may return false on Android 12+ due to scoped storage,
+        // but the ContentProvider can still serve the file to other apps. (#3565)
 
         Intent sendIntent = new Intent();
         sendIntent.setAction(intentAction);

Original file line number	Diff line number	Diff line change
`@@ -81,10 +81,12 @@ public void onReceive(Context context, Intent intent) {`
`81`	`81`	`}`
`82`	`82`
`83`	`83`	`final File fileToShare = new File(filePath);`
`84`		`- if (!(fileToShare.isFile() && fileToShare.canRead())) {`
`85`		`- Logger.logError(LOG_TAG, "Not a readable file: '" + fileToShare.getAbsolutePath() + "'");`
	`84`	`+ if (!fileToShare.isFile()) {`
	`85`	`+ Logger.logError(LOG_TAG, "Not a file: '" + fileToShare.getAbsolutePath() + "'");`
`86`	`86`	`return;`
`87`	`87`	`}`
	`88`	`+ // Note: canRead() may return false on Android 12+ due to scoped storage,`
	`89`	`+ // but the ContentProvider can still serve the file to other apps. (#3565)`
`88`	`90`
`89`	`91`	`Intent sendIntent = new Intent();`
`90`	`92`	`sendIntent.setAction(intentAction);`