summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael W <baddaemon87@gmail.com>2016-10-22 14:47:09 +0200
committerGerrit Code Review <gerrit@cyanogenmod.org>2016-11-04 13:15:06 -0700
commit537e5226c4f392c6ce0531882822b04db621d071 (patch)
tree03544fec7a568d3475b8bb83ba7839b790f117f9
parentbeec60c940e6a50046d4c7c469b9381492f8b77a (diff)
downloadandroid_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
-rw-r--r--src/com/android/camera/PhotoModule.java6
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;