search

watery01 / libyuv

Automatically exported from code.google.com/p/libyuv
0 stars 0 forks source link

valgrind fail on webrtc scale test #292

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
14:54:39 memcheck_analyze.py [ERROR] Command: 
src/out/Release/common_video_unittests
InvalidRead
Invalid read of size 2
  ScaleFilterCols_SSSE3 (third_party/libyuv/source/scale_posix.cc:913)
  ScalePlaneBilinearUp (third_party/libyuv/source/scale.cc:661)
  ScalePlane (third_party/libyuv/source/scale.cc:793)
  I420Scale (third_party/libyuv/source/scale.cc:829)
  webrtc::Scaler::Scale(webrtc::I420VideoFrame const&, webrtc::I420VideoFrame*) (webrtc/common_video/libyuv/scaler.cc:76)
  webrtc::TestScaler::ScaleSequence(webrtc::ScaleMethod, _IO_FILE*, std::basic_string<char, std::char_traits<char>, std::allocator<char> >, int, int, int, int) (webrtc/common_video/libyuv/scaler_unittest.cc:463)
  webrtc::TestScaler_BiLinearScaleTest_Test::TestBody() (webrtc/common_video/libyuv/scaler_unittest.cc:222)
Address 0x6abb456 is 25,414 bytes inside a block of size 25,415 alloc'd
  malloc (m_replacemalloc/vg_replace_malloc.c:270)
  webrtc::AlignedMalloc(unsigned long, unsigned long) (webrtc/system_wrappers/source/aligned_malloc.cc:65)
  unsigned char* webrtc::AlignedMalloc<unsigned char>(unsigned long, unsigned long) (webrtc/system_wrappers/interface/aligned_malloc.h:48)
  webrtc::Plane::MaybeResize(int) (webrtc/common_video/plane.cc:46)
  webrtc::Plane::Copy(int, int, unsigned char const*) (webrtc/common_video/plane.cc:66)
  webrtc::I420VideoFrame::CreateFrame(int, unsigned char const*, int, unsigned char const*, int, unsigned char const*, int, int, int, int, int) (webrtc/common_video/i420_video_frame.cc:53)
  webrtc::TestScaler::ScaleSequence(webrtc::ScaleMethod, _IO_FILE*, std::basic_string<char, std::char_traits<char>, std::allocator<char> >, int, int, int, int) (webrtc/common_video/libyuv/scaler_unittest.cc:460)
  webrtc::TestScaler_BiLinearScaleTest_Test::TestBody() (webrtc/common_video/libyuv/scaler_unittest.cc:222)
Suppression (error hash=#92E2EF3EA779A388#):
  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:ScaleFilterCols_SSSE3
   fun:ScalePlaneBilinearUp
   fun:ScalePlane
   fun:I420Scale
   fun:_ZN6webrtc6Scaler5ScaleERKNS_14I420VideoFrameEPS1_
   fun:_ZN6webrtc10TestScaler13ScaleSequenceENS_11ScaleMethodEP8_IO_FILESsiiii
   fun:_ZN6webrtc33TestScaler_BiLinearScaleTest_Test8TestBodyEv
}

Original issue reported on code.google.com by fbarch...@google.com on 8 Dec 2013 at 6:20

GoogleCodeExporter commented 9 years ago 
r902 of libyuv appears to resolve the valgrind issue on webrtc.

Original comment by fbarch...@google.com on 8 Dec 2013 at 7:13

GoogleCodeExporter commented 9 years ago 
There is also a problem on vertical.  When max is hit, stride still steps thru 
source.

Original comment by fbarch...@google.com on 8 Dec 2013 at 8:47

GoogleCodeExporter commented 9 years ago 
r905 addresses a vertical step of pointer.  Marking as fixed, as there are no 
reproducible issues with r905.

Original comment by fbarch...@google.com on 8 Dec 2013 at 9:20