diff options
author | Marco Nelissen <marcone@google.com> | 2016-09-30 15:21:35 -0700 |
---|---|---|
committer | Marco Nelissen <marcone@google.com> | 2016-09-30 15:21:35 -0700 |
commit | 89734e19f45f19a5c4eddfe02bde901bdecd5188 (patch) | |
tree | 3c2e9740802cadf1882d139421d3d7d5bd7c1556 | |
parent | d89181d1feb8c0fc198424d77c8275e7cab7b97c (diff) | |
download | MediaProvider-89734e19f45f19a5c4eddfe02bde901bdecd5188.tar.gz |
Throw correct exception for null path insertandroid-cts_7.1_r1android-cts-7.1_r9android-cts-7.1_r8android-cts-7.1_r7android-cts-7.1_r6android-cts-7.1_r5android-cts-7.1_r4android-cts-7.1_r3android-cts-7.1_r29android-cts-7.1_r28android-cts-7.1_r27android-cts-7.1_r26android-cts-7.1_r25android-cts-7.1_r24android-cts-7.1_r23android-cts-7.1_r22android-cts-7.1_r21android-cts-7.1_r20android-cts-7.1_r2android-cts-7.1_r19android-cts-7.1_r18android-cts-7.1_r17android-cts-7.1_r16android-cts-7.1_r15android-cts-7.1_r14android-cts-7.1_r13android-cts-7.1_r12android-cts-7.1_r11android-cts-7.1_r10android-cts-7.1_r1android-7.1.1_r9android-7.1.1_r8android-7.1.1_r7android-7.1.1_r6android-7.1.1_r43android-7.1.1_r42android-7.1.1_r41android-7.1.1_r40android-7.1.1_r4android-7.1.1_r39android-7.1.1_r38android-7.1.1_r35android-7.1.1_r33android-7.1.1_r32android-7.1.1_r31android-7.1.1_r3android-7.1.1_r28android-7.1.1_r27android-7.1.1_r26android-7.1.1_r25android-7.1.1_r24android-7.1.1_r23android-7.1.1_r22android-7.1.1_r21android-7.1.1_r20android-7.1.1_r2android-7.1.1_r17android-7.1.1_r16android-7.1.1_r15android-7.1.1_r14android-7.1.1_r13android-7.1.1_r12android-7.1.1_r11android-7.1.1_r10android-7.1.1_r1nougat-mr1.4-releasenougat-mr1.3-releasenougat-mr1.2-releasenougat-mr1-releasenougat-mr1-devnougat-mr1-cts-release
insert should return IllegalArgumentException instead of NullPointerException
Bug: 31861145
Change-Id: I12005c9ca4ebd4ab35cebdc80352c1696f4e57d1
-rw-r--r-- | src/com/android/providers/media/MediaProvider.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/com/android/providers/media/MediaProvider.java b/src/com/android/providers/media/MediaProvider.java index aa8cd888b..e181e599c 100644 --- a/src/com/android/providers/media/MediaProvider.java +++ b/src/com/android/providers/media/MediaProvider.java @@ -3214,7 +3214,7 @@ public class MediaProvider extends ContentProvider { format = MediaFile.getFormatCode(path, mimeType); } } - if (path.endsWith("/")) { + if (path != null && path.endsWith("/")) { Log.e(TAG, "directory has trailing slash: " + path); return 0; } |