OSDN Git Service

Fix crash in UpdateDisplayConfig.
authorKalyan Kondapally <kalyan.kondapally@intel.com>
Sat, 14 Apr 2018 07:11:34 +0000 (00:11 -0700)
committerKalyan Kondapally <kalyan.kondapally@intel.com>
Wed, 18 Apr 2018 03:09:01 +0000 (20:09 -0700)
We tried to access modes_, without checking if it was empty
or not. Just return in case modes are empty.

Jira: None.
Test: No crash running Media cts tests on Android IA.

wsi/drm/drmdisplay.cpp

index 8c15f60..218baaf 100644 (file)
@@ -284,6 +284,10 @@ bool DrmDisplay::GetDisplayName(uint32_t *size, char *name) {
 void DrmDisplay::UpdateDisplayConfig() {
   // update the activeConfig
   SPIN_LOCK(display_lock_);
+  if (modes_.empty()) {
+    SPIN_UNLOCK(display_lock_);
+    return;
+  }
   flags_ |= DRM_MODE_ATOMIC_ALLOW_MODESET;
   SetDisplayAttribute(modes_[config_]);
   SPIN_UNLOCK(display_lock_);