<div dir="ltr">Thanks, pushed<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 27, 2014 at 10:20 AM, Satoshi Nakagawa <span dir="ltr"><<a href="mailto:nakagawa424@oki.com" target="_blank">nakagawa424@oki.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"># HG changeset patch<br>
# User Satoshi Nakagawa <<a href="mailto:nakagawa424@oki.com">nakagawa424@oki.com</a>><br>
# Date 1414385068 -32400<br>
#      Mon Oct 27 13:44:28 2014 +0900<br>
# Node ID c1a8f6edab8f15d97e48ed872e2ae86456153e9b<br>
# Parent  67d73bffd1fdaff3df673fe10b315d26b8464c5f<br>
cugeom: fix uninitialized reported by valgrind<br>
<br>
diff -r 67d73bffd1fd -r c1a8f6edab8f source/common/cudata.h<br>
--- a/source/common/cudata.h    Sat Oct 25 22:16:39 2014 -0500<br>
+++ b/source/common/cudata.h    Mon Oct 27 13:44:28 2014 +0900<br>
@@ -76,8 +76,8 @@<br>
         SPLIT           = 1<<4, // CU is currently split in four child CUs.<br>
     };<br>
<br>
-    // (1 + 4 + 16 + 64) + (1 + 8 + 1 + 8 + 1) = 104.<br>
-    enum { MAX_GEOMS = 104 };<br>
+    // (1 + 4 + 16 + 64) = 85.<br>
+    enum { MAX_GEOMS = 85 };<br>
<br>
     uint32_t log2CUSize;    // Log of the CU size.<br>
     uint32_t childOffset;   // offset of the first child CU from current CU<br>
diff -r 67d73bffd1fd -r c1a8f6edab8f source/encoder/frameencoder.cpp<br>
--- a/source/encoder/frameencoder.cpp   Sat Oct 25 22:16:39 2014 -0500<br>
+++ b/source/encoder/frameencoder.cpp   Mon Oct 27 13:44:28 2014 +0900<br>
@@ -150,11 +150,12 @@<br>
     if (!m_cuGeoms || !m_ctuGeomMap)<br>
         return false;<br>
<br>
+    CUGeom cuLocalData[CUGeom::MAX_GEOMS];<br>
+    memset(cuLocalData, 0, sizeof(cuLocalData)); // temporal fix for memcmp<br>
+<br>
     int countGeoms = 0;<br>
     for (uint32_t ctuAddr = 0; ctuAddr < m_numRows * m_numCols; ctuAddr++)<br>
     {<br>
-        CUGeom cuLocalData[CUGeom::MAX_GEOMS];<br>
-<br>
         /* TODO: detach this logic from TComDataCU */<br>
         encData.m_picCTU[ctuAddr].initCTU(*m_frame, ctuAddr, 0);<br>
         encData.m_picCTU[ctuAddr].calcCTUGeoms(m_param->sourceWidth, m_param->sourceHeight, m_param->maxCUSize, cuLocalData);<br>
_______________________________________________<br>
x265-devel mailing list<br>
<a href="mailto:x265-devel@videolan.org">x265-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/x265-devel" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><br>
</blockquote></div><br></div>