diff options
author | Mattias Falk <mattias.falk@sonymobile.com> | 2013-01-03 12:49:09 +0100 |
---|---|---|
committer | Henrik Baard <henrik.baard@sonymobile.com> | 2013-05-02 10:40:06 +0200 |
commit | b1bf9a2a0cd929130ee75a29b5f7a9f01901a0d7 (patch) | |
tree | b5679b2cad75df9c64e621d9ba5f2f946b569662 | |
parent | 8b9cad519ef72ed4555e9dd32ecee85c922abc32 (diff) | |
download | android_packages_providers_DownloadProvider-b1bf9a2a0cd929130ee75a29b5f7a9f01901a0d7.tar.gz android_packages_providers_DownloadProvider-b1bf9a2a0cd929130ee75a29b5f7a9f01901a0d7.tar.bz2 android_packages_providers_DownloadProvider-b1bf9a2a0cd929130ee75a29b5f7a9f01901a0d7.zip |
Avoid NullPointerException
A crash will occur when a user share more
than one file and the mime type of any of
the files to share is null.
Avoid crash by not adding the mime type to
the mime type list if the value is null.
Change-Id: Ia06f3389da6ce34e11ffcf349a10fdbe64cfc9bd
-rw-r--r-- | ui/src/com/android/providers/downloads/ui/DownloadList.java | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/ui/src/com/android/providers/downloads/ui/DownloadList.java b/ui/src/com/android/providers/downloads/ui/DownloadList.java index 29ea60eb..db86ab7e 100644 --- a/ui/src/com/android/providers/downloads/ui/DownloadList.java +++ b/ui/src/com/android/providers/downloads/ui/DownloadList.java @@ -765,7 +765,9 @@ public class DownloadList extends Activity { // are all prefixes of the given mimetypes the same? ArrayList<String> mimeTypePrefixes = new ArrayList<String>(); for (String s : mimeTypes) { - mimeTypePrefixes.add(s.substring(0, s.indexOf('/'))); + if (s != null) { + mimeTypePrefixes.add(s.substring(0, s.indexOf('/'))); + } } str = findCommonString(mimeTypePrefixes); if (str != null) { |