commit 6786e5d50eedc4865b193117e6475967de711e79
author Naseer Ahmed <naseer@codeaurora.org> Fri Jun 24 16:38:52 2016 -0400
committer Naseer Ahmed <naseer@codeaurora.org> Fri Jun 24 16:43:53 2016 -0400
tree 39f09cb6845eabee68def023488d66bba71d94ba
parent 33abf3a92e7f85fb31214605efb91e0607f908cf

hwc2: Fix incorrect index

Loop from [0 to numRects)

Change-Id: I7d8651dc2c17e84a7d9c09286c36b638e5f187ac
CRs-Fixed: 1034144

sdm/libs/hwc2/hwc_layers.cpp

diff --git a/sdm/libs/hwc2/hwc_layers.cpp b/sdm/libs/hwc2/hwc_layers.cpp
index ba8c442..d35523d 100644
--- a/sdm/libs/hwc2/hwc_layers.cpp
+++ b/sdm/libs/hwc2/hwc_layers.cpp
@@ -93,14 +93,11 @@
 }
 
 HWC2::Error HWCLayer::SetLayerSurfaceDamage(hwc_region_t damage) {
-  auto num_dirty_rects = damage.numRects;
   layer_->dirty_regions.clear();
-  if (num_dirty_rects > 0) {
-    for (uint32_t i = 0; i <= damage.numRects; i++) {
-      LayerRect rect;
-      SetRect(damage.rects[i], &rect);
-      layer_->dirty_regions.push_back(rect);
-    }
+  for (uint32_t i = 0; i < damage.numRects; i++) {
+    LayerRect rect;
+    SetRect(damage.rects[i], &rect);
+    layer_->dirty_regions.push_back(rect);
   }
   return HWC2::Error::None;
 }
@@ -242,14 +239,11 @@
 }
 
 HWC2::Error HWCLayer::SetLayerVisibleRegion(hwc_region_t visible) {
-  auto num_dirty_rects = visible.numRects;
   layer_->visible_regions.clear();
-  if (num_dirty_rects > 0) {
-    for (uint32_t i = 0; i <= visible.numRects; i++) {
-      LayerRect rect;
-      SetRect(visible.rects[i], &rect);
-      layer_->visible_regions.push_back(rect);
-    }
+  for (uint32_t i = 0; i < visible.numRects; i++) {
+    LayerRect rect;
+    SetRect(visible.rects[i], &rect);
+    layer_->visible_regions.push_back(rect);
   }
 
   return HWC2::Error::None;