shithub: openh264

Download patch

ref: 0298b3f5807a132e837fc86f65a8f6839d647b00
parent: f76325edc75f031edba17266e744f6920ebadffc
author: Martin Storsjö <martin@martin.st>
date: Wed Jun 3 05:45:06 EDT 2015

Initialize enough samples in the new 4x8 tests

This fixes valgrind warnings about tests using uninitialized data.

--- a/test/encoder/EncUT_Sample.cpp
+++ b/test/encoder/EncUT_Sample.cpp
@@ -207,9 +207,9 @@
 }
 
 TEST_F (SadSatdCFuncTest, WelsSampleSad4x8_c) {
-  for (int i = 0; i < (m_iStrideA << 2); i++)
+  for (int i = 0; i < (m_iStrideA << 3); i++)
     m_pPixSrcA[i] = rand() % 256;
-  for (int i = 0; i < (m_iStrideB << 2); i++)
+  for (int i = 0; i < (m_iStrideB << 3); i++)
     m_pPixSrcB[i] = rand() % 256;
   uint8_t* pPixA = m_pPixSrcA;
   uint8_t* pPixB = m_pPixSrcB;
@@ -504,9 +504,9 @@
 }
 
 TEST_F (SadSatdCFuncTest, WelsSampleSadFour4x8_c) {
-  for (int i = 0; i < (m_iStrideA << 3); i++)
+  for (int i = 0; i < (m_iStrideA << 4); i++)
     m_pPixSrcA[i] = rand() % 256;
-  for (int i = 0; i < (m_iStrideB << 3); i++)
+  for (int i = 0; i < (m_iStrideB << 4); i++)
     m_pPixSrcB[i] = rand() % 256;
   uint8_t* pPixA = m_pPixSrcA;
   uint8_t* pPixB = m_pPixSrcB + m_iStrideB;