commit ae61b2b2c35b1d6ef87a9822ca7868f9984f11b2
author Saurabh Shah <saurshah@codeaurora.org> Wed Apr 10 16:37:25 2013 -0700
committer Gerrit - the friendly Code Review server <code-review@localhost> Mon Apr 22 17:46:01 2013 -0700
tree 7510691c46c4c710a67c83437c8438f5e683365e
parent f4598bd0cd518565e55dd9939d814f10b36798ef

hwc: Fix dumpsys crash

In extreme cases, when all pipes are used, the buffer is insufficient.
Reduce logging by striping off unnecessary bytes.

Change-Id: I5a6ffb9ff288e389eea4e9ca4ac850ed59d539ff

liboverlay/overlayUtils.cpp

diff --git a/liboverlay/overlayUtils.cpp b/liboverlay/overlayUtils.cpp
index 8849984..e595cef 100644
--- a/liboverlay/overlayUtils.cpp
+++ b/liboverlay/overlayUtils.cpp
@@ -361,9 +361,9 @@
             prefix, ov.id, ov.z_order, ov.is_fg, ov.alpha,
             ov.transp_mask, ov.flags, ov.horz_deci, ov.vert_deci);
     strncat(buf, str, strlen(str));
-    getDump(buf, len, "\tsrc(msmfb_img)", ov.src);
-    getDump(buf, len, "\tsrc_rect(mdp_rect)", ov.src_rect);
-    getDump(buf, len, "\tdst_rect(mdp_rect)", ov.dst_rect);
+    getDump(buf, len, "\tsrc", ov.src);
+    getDump(buf, len, "\tsrc_rect", ov.src_rect);
+    getDump(buf, len, "\tdst_rect", ov.dst_rect);
 }
 
 void getDump(char *buf, size_t len, const char *prefix,
@@ -392,7 +392,7 @@
             "%s id=%d\n",
             prefix, ov.id);
     strncat(buf, str, strlen(str));
-    getDump(buf, len, "\tdata(msmfb_data)", ov.data);
+    getDump(buf, len, "\tdata", ov.data);
 }
 
 void getDump(char *buf, size_t len, const char *prefix,
@@ -421,13 +421,11 @@
         const msm_rotator_data_info& rot) {
     char str[256] = {'\0'};
     snprintf(str, 256,
-            "%s sessid=%u verkey=%d\n",
-            prefix, rot.session_id, rot.version_key);
+            "%s sessid=%u\n",
+            prefix, rot.session_id);
     strncat(buf, str, strlen(str));
     getDump(buf, len, "\tsrc", rot.src);
     getDump(buf, len, "\tdst", rot.dst);
-    getDump(buf, len, "\tsrc_chroma", rot.src_chroma);
-    getDump(buf, len, "\tdst_chroma", rot.dst_chroma);
 }
 
 } // utils