commit 24a42e9f54e971a17b829e85681c68d60a178d26
author Dan Stoza <stoza@google.com> Mon Mar 09 10:04:11 2015 -0700
committer Dan Stoza <stoza@google.com> Mon Mar 09 10:04:11 2015 -0700
tree 98e28f4343362bf6f81d2062337368559ad06dfc
parent 5fec79156d749322dcc0376da1d6cf4d9ee9ccf4

SurfaceFlinger: Fix null pointer exception

Fixes a null pointer exception in getActiveConfig

Bug: 19416124
Change-Id: I45a01f67affb63a348866ce42a6013c5eb71fe8f

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 6a5a39e..1069580 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -617,7 +617,11 @@
 }
 
 int SurfaceFlinger::getActiveConfig(const sp<IBinder>& display) {
-    return getDisplayDevice(display)->getActiveConfig();
+    sp<DisplayDevice> device(getDisplayDevice(display));
+    if (device != NULL) {
+        return device->getActiveConfig();
+    }
+    return BAD_VALUE;
 }
 
 void SurfaceFlinger::setActiveConfigInternal(const sp<DisplayDevice>& hw, int mode) {