OSDN Git Service

sf: Add a NULL check in getDisplayConfigs
authorNaseer Ahmed <naseer@codeaurora.org>
Mon, 3 Nov 2014 19:49:23 +0000 (14:49 -0500)
committerJesse Hall <jessehall@google.com>
Mon, 3 Nov 2014 20:16:18 +0000 (12:16 -0800)
commit7aa0c47e4205c6fca136c38f272d911c25c8a8fa
tree121a1180e9797e633ded857f5a4f5949f9e5d5c6
parent793fc0e13d25bdecda1219999f8be2cb3c121d20
sf: Add a NULL check in getDisplayConfigs

Validate the display binder by adding a NULL check in getDisplayConfigs.
This will prevent a false match if the caller queries the display
configs for an inactive display (whose binder is NULL by default).

Without this change we might end up attempting to index the display
config array, which is unpopulated for inactive displays, and this will
result in a crash. (See getDisplayInfo in SurfaceComposerClient.cpp for
an example of this scenario)

Change-Id: I1a12f43b7c375b9c01998dadd5b658275c733fb2
Acked-by: Tatenda Chipeperekwa <tatendac@codeaurora.org>
services/surfaceflinger/SurfaceFlinger.cpp