commit 1c5431e111e48be8cfc07c8d64ce2ee58c3bc123
author Praveena Pachipulusu <pveena@codeaurora.org> Fri May 29 19:38:30 2015 +0530
committer Gerrit - the friendly Code Review server <code-review@localhost> Sun May 31 21:47:51 2015 -0700
tree 6c7598bf4cba5f1230bb3ce05b6213825ead1728
parent 9b6b7f75376fac3360c6c3a72359b0b362670914

hwc: Reset the refresh rate alone to avoid stack corruption.

In early return calls of prepare, reset the refresh rate
irrespective of layer properties to avoid stack corruption.

Change-Id: I5a04e111b1f10a6cb807459c7693d049c454edaa
CRs-Fixed: 838902

libhwcomposer/hwc_mdpcomp.cpp

diff --git a/libhwcomposer/hwc_mdpcomp.cpp b/libhwcomposer/hwc_mdpcomp.cpp
index 3e3638c..5949f27 100644
--- a/libhwcomposer/hwc_mdpcomp.cpp
+++ b/libhwcomposer/hwc_mdpcomp.cpp
@@ -2181,7 +2181,8 @@
                 __FUNCTION__);
         mCachedFrame.reset();
 #ifdef DYNAMIC_FPS
-        setDynRefreshRate(ctx, list);
+        // Reset refresh rate
+        setRefreshRate(ctx, mDpy, ctx->dpyAttr[mDpy].refreshRate);
 #endif
         freeHwCursor(ctx->dpyAttr[mDpy].fd, mDpy);
         return -1;
@@ -2198,7 +2199,8 @@
         setMDPCompLayerFlags(ctx, list);
         mCachedFrame.updateCounts(mCurrentFrame);
 #ifdef DYNAMIC_FPS
-        setDynRefreshRate(ctx, list);
+        // Reset refresh rate
+        setRefreshRate(ctx, mDpy, ctx->dpyAttr[mDpy].refreshRate);
 #endif
         freeHwCursor(ctx->dpyAttr[mDpy].fd, mDpy);
         ret = -1;