diff options
author | Michael W <baddaemon87@gmail.com> | 2016-10-22 14:47:09 +0200 |
---|---|---|
committer | Gerrit Code Review <gerrit@cyanogenmod.org> | 2016-11-04 13:15:06 -0700 |
commit | 537e5226c4f392c6ce0531882822b04db621d071 (patch) | |
tree | 03544fec7a568d3475b8bb83ba7839b790f117f9 /src/com/android/camera | |
parent | beec60c940e6a50046d4c7c469b9381492f8b77a (diff) | |
download | android_packages_apps_Snap-537e5226c4f392c6ce0531882822b04db621d071.tar.gz android_packages_apps_Snap-537e5226c4f392c6ce0531882822b04db621d071.tar.bz2 android_packages_apps_Snap-537e5226c4f392c6ce0531882822b04db621d071.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
Diffstat (limited to 'src/com/android/camera')
-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 e12ffc5a2..33892ddba 100644 --- a/src/com/android/camera/PhotoModule.java +++ b/src/com/android/camera/PhotoModule.java @@ -4615,8 +4615,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; |