diff options
author | Michael W <baddaemon87@gmail.com> | 2016-10-22 14:47:09 +0200 |
---|---|---|
committer | Daniel Hillenbrand <codeworkx@cyanogenmod.org> | 2016-11-04 13:17:54 -0700 |
commit | 7c9beb765b60e156202175cc3840d0f7c36979f3 (patch) | |
tree | d43dc6557c381f0e3068ecc94b5f64ba6e8dce77 | |
parent | 95309463d1f17c6d5ba78d31817f70a45c9624ff (diff) | |
download | android_packages_apps_Snap-7c9beb765b60e156202175cc3840d0f7c36979f3.tar.gz android_packages_apps_Snap-7c9beb765b60e156202175cc3840d0f7c36979f3.tar.bz2 android_packages_apps_Snap-7c9beb765b60e156202175cc3840d0f7c36979f3.zip |
Snap: Fix possible NPE
Seems like getSupportedFocusModes() can return null.
Check for null before using it further
Change-Id: I46860f780b99970cbe9857c8ba73691d69321488
Reference: BugDumps 20161014-20 L#131
(cherry picked from commit 537e5226c4f392c6ce0531882822b04db621d071)
-rw-r--r-- | src/com/android/camera/PhotoModule.java | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/com/android/camera/PhotoModule.java b/src/com/android/camera/PhotoModule.java index 7f08d78f4..7cad02919 100644 --- a/src/com/android/camera/PhotoModule.java +++ b/src/com/android/camera/PhotoModule.java @@ -4621,8 +4621,10 @@ public class PhotoModule mAeLockSupported = CameraUtil.isAutoExposureLockSupported(mInitialParams); mAwbLockSupported = CameraUtil.isAutoWhiteBalanceLockSupported(mInitialParams); - if (mInitialParams.getSupportedFocusModes().contains(CameraUtil.FOCUS_MODE_CONTINUOUS_PICTURE) || - mInitialParams.getSupportedFocusModes().contains(CameraUtil.FOCUS_MODE_MW_CONTINUOUS_PICTURE)) { + List<String> focusModes = mInitialParams.getSupportedFocusModes(); + if (focusModes != null && + (focusModes.contains(CameraUtil.FOCUS_MODE_CONTINUOUS_PICTURE) || + focusModes.contains(CameraUtil.FOCUS_MODE_MW_CONTINUOUS_PICTURE))) { mContinuousFocusSupported = true; } else { mContinuousFocusSupported = false; |