summaryrefslogtreecommitdiff
path: root/src/com/android/contacts/util/ContactPhotoUtils.java
diff options
context:
space:
mode:
Diffstat (limited to 'src/com/android/contacts/util/ContactPhotoUtils.java')
-rw-r--r--src/com/android/contacts/util/ContactPhotoUtils.java22
1 files changed, 18 insertions, 4 deletions
diff --git a/src/com/android/contacts/util/ContactPhotoUtils.java b/src/com/android/contacts/util/ContactPhotoUtils.java
index ce691c3b7..a80af7f20 100644
--- a/src/com/android/contacts/util/ContactPhotoUtils.java
+++ b/src/com/android/contacts/util/ContactPhotoUtils.java
@@ -18,19 +18,17 @@
package com.android.contacts.util;
import android.content.ClipData;
+import android.content.ContentResolver;
import android.content.Context;
import android.content.Intent;
import android.graphics.Bitmap;
import android.graphics.BitmapFactory;
import android.net.Uri;
-import android.os.Environment;
import android.provider.MediaStore;
import android.support.v4.content.FileProvider;
import android.util.Log;
-
import com.android.contacts.R;
import com.google.common.io.Closeables;
-
import java.io.ByteArrayOutputStream;
import java.io.File;
import java.io.FileNotFoundException;
@@ -148,7 +146,7 @@ public class ContactPhotoUtils {
*/
public static boolean savePhotoFromUriToUri(Context context, Uri inputUri, Uri outputUri,
boolean deleteAfterSave) {
- if (inputUri == null || outputUri == null) {
+ if (inputUri == null || outputUri == null || isFilePathAndNotStorage(inputUri)) {
return false;
}
try (FileOutputStream outputStream = context.getContentResolver()
@@ -173,4 +171,20 @@ public class ContactPhotoUtils {
}
return true;
}
+
+ /**
+ * Returns {@code true} if the {@code inputUri} is a FILE scheme and it does not point to
+ * the storage directory.
+ */
+ private static boolean isFilePathAndNotStorage(Uri inputUri) {
+ if (ContentResolver.SCHEME_FILE.equals(inputUri.getScheme())) {
+ try {
+ File file = new File(inputUri.getPath()).getCanonicalFile();
+ return !file.getCanonicalPath().startsWith("/storage/");
+ } catch (IOException e) {
+ return false;
+ }
+ }
+ return false;
+ }
}