From 5b03b40f1d99de196e336ff0506889314de60446 Mon Sep 17 00:00:00 2001
From: M66B <M66B@users.noreply.github.com>
Date: Sat, 23 Jan 2021 10:26:05 +0100
Subject: [PATCH] Santize shared attachments

---
 .../java/eu/faircode/email/ActivityCompose.java     | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/app/src/main/java/eu/faircode/email/ActivityCompose.java b/app/src/main/java/eu/faircode/email/ActivityCompose.java
index 63667e7121..a85a73acbc 100644
--- a/app/src/main/java/eu/faircode/email/ActivityCompose.java
+++ b/app/src/main/java/eu/faircode/email/ActivityCompose.java
@@ -171,9 +171,16 @@ public class ActivityCompose extends ActivityBase implements FragmentManager.OnB
 
             if (intent.hasExtra(Intent.EXTRA_STREAM))
                 if (Intent.ACTION_SEND_MULTIPLE.equals(action)) {
-                    ArrayList<Uri> uris = intent.getParcelableArrayListExtra(Intent.EXTRA_STREAM);
-                    if (uris != null)
-                        args.putParcelableArrayList("attachments", uris);
+                    ArrayList<Uri> streams = intent.getParcelableArrayListExtra(Intent.EXTRA_STREAM);
+                    if (streams != null) {
+                        // Some apps send null streams
+                        ArrayList<Uri> uris = new ArrayList<>();
+                        for (Uri stream : streams)
+                            if (stream != null)
+                                uris.add(stream);
+                        if (uris.size() > 0)
+                            args.putParcelableArrayList("attachments", uris);
+                    }
                 } else {
                     Uri stream = intent.getParcelableExtra(Intent.EXTRA_STREAM);
                     if (stream != null) {