diff options
| author | 2014-04-29 16:16:29 +0800 | |
|---|---|---|
| committer | 2014-04-30 17:09:19 +0800 | |
| commit | 1c0f21ea8475f9996dbd7400725969ec5ffdc005 (patch) | |
| tree | f802e1cd4d30eb640606aae82b5b01c863d6fea5 | |
| parent | 10d57c4bc7459d0c73b254af006a3528e9a0c33e (diff) | |
WallpaperManager: add NULL pointer check for getCropAndSetWallpaperIntent
Originally the api 'getCropAndSetWallpaperIntent' does not check
whether the parsed parameter is a NULL pointer, as leads to the
whole app crash if it call this api with a null parameter accidentally.
A null pointer check logic is added into its original parameter check
logic, so that this API will throw an IllegalArgumentException if it
is parsed with a null pointer
Change-Id: Ib7de40e571419d09e1a744edc969eb7162766b75
| -rw-r--r-- | core/java/android/app/WallpaperManager.java | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/core/java/android/app/WallpaperManager.java b/core/java/android/app/WallpaperManager.java index ced72f80a029..c2bbff0e743a 100644 --- a/core/java/android/app/WallpaperManager.java +++ b/core/java/android/app/WallpaperManager.java @@ -668,6 +668,10 @@ public class WallpaperManager { * not "image/*" */ public Intent getCropAndSetWallpaperIntent(Uri imageUri) { + if (imageUri == null) { + throw new IllegalArgumentException("Image URI must not be null"); + } + if (!ContentResolver.SCHEME_CONTENT.equals(imageUri.getScheme())) { throw new IllegalArgumentException("Image URI must be of the " + ContentResolver.SCHEME_CONTENT + " scheme type"); |