OSDN Git Service

WallpaperManager: add NULL pointer check for getCropAndSetWallpaperIntent
authorAllen Shen <ashen@nvidia.com>
Tue, 29 Apr 2014 08:16:29 +0000 (16:16 +0800)
committerAllen Shen <ashen@nvidia.com>
Wed, 30 Apr 2014 09:09:19 +0000 (17:09 +0800)
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

core/java/android/app/WallpaperManager.java

index ced72f8..c2bbff0 100644 (file)
@@ -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");