diff options
| author | Terry Heo <terryheo@google.com> | 2014-11-13 15:56:22 -0800 |
|---|---|---|
| committer | Terry (Woncheol) Heo <terryheo@google.com> | 2014-11-14 00:01:55 +0000 |
| commit | 344b2d4293049df49881664a5c4c20dcb68ae7ec (patch) | |
| tree | 99d284a5f525ce729615f3fe422182b53638dffe | |
| parent | 6331ea953c11bbea7d7e027c8fa45fb4281cb1b0 (diff) | |
| download | android_packages_providers_TvProvider-344b2d4293049df49881664a5c4c20dcb68ae7ec.tar.gz android_packages_providers_TvProvider-344b2d4293049df49881664a5c4c20dcb68ae7ec.tar.bz2 android_packages_providers_TvProvider-344b2d4293049df49881664a5c4c20dcb68ae7ec.zip | |
Throw FileNotFoundException when openLogoFile() fails
ContentProvider should throw the exception instead of returning null object.
Bug: 18335090
Change-Id: Iffa0370b6c1c901ce665ca21aac6c94239fa476f
| -rw-r--r-- | src/com/android/providers/tv/TvProvider.java | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/com/android/providers/tv/TvProvider.java b/src/com/android/providers/tv/TvProvider.java index ea8dca7..cd70ad1 100644 --- a/src/com/android/providers/tv/TvProvider.java +++ b/src/com/android/providers/tv/TvProvider.java @@ -858,7 +858,11 @@ public class TvProvider extends ContentProvider { if (mode.equals("r")) { String sql = queryBuilder.buildQuery(new String[] { CHANNELS_COLUMN_LOGO }, params.getSelection(), null, null, null, null); - return DatabaseUtils.blobFileDescriptorForQuery(db, sql, params.getSelectionArgs()); + ParcelFileDescriptor fd = DatabaseUtils.blobFileDescriptorForQuery(db, sql, params.getSelectionArgs()); + if (fd == null) { + throw new FileNotFoundException(uri.toString()); + } + return fd; } else { try (Cursor cursor = queryBuilder.query(db, new String[] { Channels._ID }, params.getSelection(), params.getSelectionArgs(), null, null, null)) { |