KerwinMa / libyuv

Automatically exported from code.google.com/p/libyuv
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

YUY2ToARGB overread - valgrind #149

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Invalid read of size 8
  YUY2ToUV422Row_Unaligned_SSE2 (/source/row_posix.cc:3031)
  YUY2ToARGBRow_Unaligned_SSSE3 (/source/row_common.cc:1582)
  YUY2ToARGBRow_Any_SSSE3 (/source/row_any.cc:145)
  YUY2ToARGB (/source/convert_argb.cc:804)
  libyuv::libyuvTest_YUY2ToARGB_Random_Test::TestBody() (/unit_test/convert_test.cc:825)
Address 0x6819d0c is 4,092 bytes inside a block of size 4,096 alloc'd
  malloc (m_replacemalloc/vg_replace_malloc.c:263)
  libyuv::libyuvTest_YUY2ToARGB_Random_Test::TestBody() (/unit_test/convert_test.cc:825)
Suppression (error hash=#714B719EAAFC765D#):
  For more info on using suppressions see http://dev.chromium.org/developers/tree-sheriffs/sheriff-details-chromium/memory-sheriff#TOC-Suppressing-memory-reports
{
   <insert_a_suppression_name_here>
   Memcheck:Unaddressable
   fun:YUY2ToUV422Row_Unaligned_SSE2
   fun:YUY2ToARGBRow_Unaligned_SSSE3
   fun:YUY2ToARGBRow_Any_SSSE3
   fun:YUY2ToARGB
   fun:_ZN6libyuv33libyuvTest_YUY2ToARGB_Random_Test8TestBodyEv
}

Original issue reported on code.google.com by fbarch...@google.com on 6 Nov 2012 at 6:46

GoogleCodeExporter commented 9 years ago
[----------] Global test environment tear-down
[==========] 5 tests from 1 test case ran. (631 ms total)
[  PASSED  ] 2 tests.
[  FAILED  ] 3 tests, listed below:
[  FAILED  ] libyuvTest.ARGBToYUY2_Any
[  FAILED  ] libyuvTest.ARGBToYUY2_Unaligned
[  FAILED  ] libyuvTest.ARGBToYUY2_Invert

Original comment by fbarch...@google.com on 7 Nov 2012 at 8:56

GoogleCodeExporter commented 9 years ago
r481 ARGBToYUY2 uses UV422
unittest alignes allocation width to multiple of 4 bytes for YUY2 buffers.

Original comment by fbarch...@chromium.org on 12 Nov 2012 at 10:37