commit 763e39ad54cc231f10c71722c2024654320e123d
author Steve Kondik <steve@cyngn.com> Wed Apr 13 18:14:10 2016 -0700
committer Gerrit Code Review <gerrit@cyanogenmod.org> Thu Apr 14 20:34:59 2016 -0700
tree 179d4cbdeec0d5c3dd42fac3724c7fb3974dbfc3
parent 14190e4f00113d7fc69d127259634673c82c5eed

livedisplay: Fix error checks when fetching config

Change-Id: I8ba43f32b2c45f626401f3cbc0050bc0593748c8

sdk/src/java/cyanogenmod/hardware/LiveDisplayManager.java

diff --git a/sdk/src/java/cyanogenmod/hardware/LiveDisplayManager.java b/sdk/src/java/cyanogenmod/hardware/LiveDisplayManager.java
index 0e4f7a1..e75c9d8 100644
--- a/sdk/src/java/cyanogenmod/hardware/LiveDisplayManager.java
+++ b/sdk/src/java/cyanogenmod/hardware/LiveDisplayManager.java
@@ -127,8 +127,8 @@
         }
         sService = getService();
 
-        if (context.getPackageManager().hasSystemFeature(
-                CMContextConstants.Features.LIVEDISPLAY) && !checkService()) {
+        if (!context.getPackageManager().hasSystemFeature(
+                CMContextConstants.Features.LIVEDISPLAY) || !checkService()) {
             throw new RuntimeException("Unable to get LiveDisplayService. The service either" +
                     " crashed, was not started, or the interface has been called to early in" +
                     " SystemServer init");
@@ -136,6 +136,9 @@
 
         try {
             mConfig = sService.getConfig();
+            if (mConfig == null) {
+                throw new RuntimeException("Unable to get LiveDisplay configuration!");
+            }
         } catch (RemoteException e) {
             throw new RuntimeException("Unable to fetch LiveDisplay configuration!", e);
         }