foolab / libyuv

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

valigrind chrome fail on I420ToARGB #289

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
[----------] 3 tests from YuvToRgbTest
[ RUN      ] YuvToRgbTest.BasicConversion
[       OK ] YuvToRgbTest.BasicConversion (47 ms)
[ RUN      ] YuvToRgbTest.Clipping
[       OK ] YuvToRgbTest.Clipping (111 ms)
[ RUN      ] YuvToRgbTest.ClippingAndScaling
[       OK ] YuvToRgbTest.ClippingAndScaling (13 ms)
[----------] 3 tests from YuvToRgbTest (172 ms total)

08:35:08 memcheck_analyze.py [ERROR] Command: 
/mnt/data/b/build/slave/chromium-rel-chromeos-valgrind-tests-2/build/src/out/Rel
ease/remoting_unittests --gtest_print_time 
--gtest_filter=-VideoSchedulerTest.FAILS_StartAndStop:VideoSchedulerTest.StartAn
dStop:VideoSchedulerTest.FLAKY_StartAndStop --test-tiny-timeout=1000 
--ui-test-action-timeout=60000 --ui-test-action-max-timeout=150000
InvalidRead
Invalid read of size 8
  I422ToARGBRow_Unaligned_SSSE3 (third_party/libyuv/source/row_posix.cc:2631)
  I422ToARGBRow_Any_SSSE3 (third_party/libyuv/source/row_any.cc:40)
  I420ToARGB (third_party/libyuv/source/convert_from.cc:587)
  remoting::ConvertAndScaleYUVToRGB32Rect(unsigned char const*, unsigned char const*, unsigned char const*, int, int, webrtc::DesktopSize const&, webrtc::DesktopRect const&, unsigned char*, int, webrtc::DesktopSize const&, webrtc::DesktopRect const&, webrtc::DesktopRect const&) (remoting/base/util.cc:132)
  remoting::YuvToRgbTester::RunTest(webrtc::DesktopSize, webrtc::DesktopRect const&) (remoting/base/util_unittest.cc:119)
  remoting::YuvToRgbTest_Clipping_Test::TestBody() (remoting/base/util_unittest.cc:162)
Address 0xd247099 is 1,145 bytes inside a block of size 1,152 alloc'd
  operator new[](unsigned long) (m_replacemalloc/vg_replace_malloc.c:1144)
  remoting::YuvToRgbTester::YuvToRgbTester() (remoting/base/util_unittest.cc:25)
  remoting::YuvToRgbTest_Clipping_Test::TestBody() (remoting/base/util_unittest.cc:150)
Suppression (error hash=#5738D70435913705#):
  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:I422ToARGBRow_Unaligned_SSSE3
   fun:I422ToARGBRow_Any_SSSE3
   fun:I420ToARGB
   fun:_ZN8remoting29ConvertAndScaleYUVToRGB32RectEPKhS1_S1_iiRKN6webrtc11DesktopSizeERKNS2_11DesktopRectEPhiS5_S8_S8_
   fun:_ZN8remoting14YuvToRgbTester7RunTestEN6webrtc11DesktopSizeERKNS1_11DesktopRectE
   fun:_ZN8remoting26YuvToRgbTest_Clipping_Test8TestBodyEv
}

08:35:08 memcheck_analyze.py [ERROR] Command: 
/mnt/data/b/build/slave/chromium-rel-chromeos-valgrind-tests-2/build/src/out/Rel
ease/remoting_unittests --gtest_print_time 
--gtest_filter=-VideoSchedulerTest.FAILS_StartAndStop:VideoSchedulerTest.StartAn
dStop:VideoSchedulerTest.FLAKY_StartAndStop --test-tiny-timeout=1000 
--ui-test-action-timeout=60000 --ui-test-action-max-timeout=150000
InvalidRead
Invalid read of size 8
  I422ToARGBRow_SSSE3 (third_party/libyuv/source/row_posix.cc:2446)
  I420ToARGB (third_party/libyuv/source/convert_from.cc:587)
  remoting::ConvertAndScaleYUVToRGB32Rect(unsigned char const*, unsigned char const*, unsigned char const*, int, int, webrtc::DesktopSize const&, webrtc::DesktopRect const&, unsigned char*, int, webrtc::DesktopSize const&, webrtc::DesktopRect const&, webrtc::DesktopRect const&) (remoting/base/util.cc:132)
  remoting::YuvToRgbTester::RunTest(webrtc::DesktopSize, webrtc::DesktopRect const&) (remoting/base/util_unittest.cc:119)
  remoting::YuvToRgbTester::TestBasicConversion() (remoting/base/util_unittest.cc:128)
  remoting::YuvToRgbTest_BasicConversion_Test::TestBody() (remoting/base/util_unittest.cc:146)
Address 0xd245c1c is 1,148 bytes inside a block of size 1,152 alloc'd
  operator new[](unsigned long) (m_replacemalloc/vg_replace_malloc.c:1144)
  remoting::YuvToRgbTester::YuvToRgbTester() (remoting/base/util_unittest.cc:25)
  remoting::YuvToRgbTest_BasicConversion_Test::TestBody() (remoting/base/util_unittest.cc:145)
Suppression (error hash=#AA1FB6B1DC1833AB#):
  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:I422ToARGBRow_SSSE3
   fun:I420ToARGB
   fun:_ZN8remoting29ConvertAndScaleYUVToRGB32RectEPKhS1_S1_iiRKN6webrtc11DesktopSizeERKNS2_11DesktopRectEPhiS5_S8_S8_
   fun:_ZN8remoting14YuvToRgbTester7RunTestEN6webrtc11DesktopSizeERKNS1_11DesktopRectE
   fun:_ZN8remoting14YuvToRgbTester19TestBasicConversionEv
   fun:_ZN8remoting33YuvToRgbTest_BasicConversion_Test8TestBodyEv
}

08:35:08 valgrind_test.py [INFO] Please see 
http://dev.chromium.org/developers/how-tos/using-valgrind for the info on 
Memcheck/Valgrind
08:35:08 valgrind_test.py [ERROR] Analyze failed.
08:35:08 valgrind_test.py [INFO] Search the log for '[ERROR]' to see the error 
reports.
08:35:08 valgrind_test.py [INFO] elapsed time: 00:01:13
Stopping Xvfb with pid 8464 ...
Xvfb pid file removed
exit code (as seen by runtest.py): 255
@@@STEP_LOG_LINE@5738D70435913705@Suppression (error 
hash=#5738D70435913705#):@@@
@@@STEP_LOG_LINE@5738D70435913705@For more info on using suppressions see 
http://dev.chromium.org/developers/tree-sheriffs/sheriff-details-chromium/memory
-sheriff#TOC-Suppressing-memory-reports@@@
@@@STEP_LOG_LINE@5738D70435913705@{@@@
@@@STEP_LOG_LINE@5738D70435913705@<insert_a_suppression_name_here>@@@
@@@STEP_LOG_LINE@5738D70435913705@Memcheck:Unaddressable@@@
@@@STEP_LOG_LINE@5738D70435913705@fun:I422ToARGBRow_Unaligned_SSSE3@@@
@@@STEP_LOG_LINE@5738D70435913705@fun:I422ToARGBRow_Any_SSSE3@@@
@@@STEP_LOG_LINE@5738D70435913705@fun:I420ToARGB@@@
@@@STEP_LOG_LINE@5738D70435913705@fun:_ZN8remoting29ConvertAndScaleYUVToRGB32Rec
tEPKhS1_S1_iiRKN6webrtc11DesktopSizeERKNS2_11DesktopRectEPhiS5_S8_S8_@@@
@@@STEP_LOG_LINE@5738D70435913705@fun:_ZN8remoting14YuvToRgbTester7RunTestEN6web
rtc11DesktopSizeERKNS1_11DesktopRectE@@@
@@@STEP_LOG_LINE@5738D70435913705@fun:_ZN8remoting26YuvToRgbTest_Clipping_Test8T
estBodyEv@@@
@@@STEP_LOG_LINE@5738D70435913705@}@@@
@@@STEP_LOG_END@5738D70435913705@@@
@@@STEP_LOG_LINE@AA1FB6B1DC1833AB@Suppression (error 
hash=#AA1FB6B1DC1833AB#):@@@
@@@STEP_LOG_LINE@AA1FB6B1DC1833AB@For more info on using suppressions see 
http://dev.chromium.org/developers/tree-sheriffs/sheriff-details-chromium/memory
-sheriff#TOC-Suppressing-memory-reports@@@
@@@STEP_LOG_LINE@AA1FB6B1DC1833AB@{@@@
@@@STEP_LOG_LINE@AA1FB6B1DC1833AB@<insert_a_suppression_name_here>@@@
@@@STEP_LOG_LINE@AA1FB6B1DC1833AB@Memcheck:Unaddressable@@@
@@@STEP_LOG_LINE@AA1FB6B1DC1833AB@fun:I422ToARGBRow_SSSE3@@@
@@@STEP_LOG_LINE@AA1FB6B1DC1833AB@fun:I420ToARGB@@@
@@@STEP_LOG_LINE@AA1FB6B1DC1833AB@fun:_ZN8remoting29ConvertAndScaleYUVToRGB32Rec
tEPKhS1_S1_iiRKN6webrtc11DesktopSizeERKNS2_11DesktopRectEPhiS5_S8_S8_@@@
@@@STEP_LOG_LINE@AA1FB6B1DC1833AB@fun:_ZN8remoting14YuvToRgbTester7RunTestEN6web
rtc11DesktopSizeERKNS1_11DesktopRectE@@@
@@@STEP_LOG_LINE@AA1FB6B1DC1833AB@fun:_ZN8remoting14YuvToRgbTester19TestBasicCon
versionEv@@@
@@@STEP_LOG_LINE@AA1FB6B1DC1833AB@fun:_ZN8remoting33YuvToRgbTest_BasicConversion
_Test8TestBodyEv@@@
@@@STEP_LOG_LINE@AA1FB6B1DC1833AB@}@@@
@@@STEP_LOG_END@AA1FB6B1DC1833AB@@@
@@@STEP_FAILURE@@@
@@@STEP_TEXT@memory test: remoting@@@
@@@STEP_TEXT@1 disabled@@@
@@@STEP_TEXT@crashed or hung@@@
 killed dbus-daemon with PID 8460
 cleared DBUS_SESSION_BUS_ADDRESS environment variable
program finished with exit code 255
elapsedTime=74.137323

Original issue reported on code.google.com by fbarch...@chromium.org on 2 Dec 2013 at 4:57

GoogleCodeExporter commented 9 years ago
May be related to odd width, which fails under valgrind and drmemory.

Original comment by fbarch...@chromium.org on 2 Dec 2013 at 7:17

GoogleCodeExporter commented 9 years ago
r885 fixes  I411, I444, I422 to I420 for odd source width.

Seeing height to odd, bayer fails valgrind:

-----------------------------------------------------
16:15:10 memcheck_analyze.py [ERROR] FAIL! There were 28 errors: 
16:15:10 memcheck_analyze.py [ERROR] Command: src/out/Release/libyuv_unittest
UninitCondition
Conditional jump or move depends on uninitialised value(s)
  testing::AssertionResult testing::internal::CmpHelperLE<int, int>(char const*, char const*, int const&, int const&) (/mnt/data/b/build/slave/linux_memcheck/build/src/out/Release/libyuv_unittest)
  libyuv::libyuvTest_BayerBGGRToARGB_Unaligned_Test::TestBody() (/mnt/data/b/build/slave/linux_memcheck/build/src/out/Release/libyuv_unittest)
Suppression (error hash=#74E1F8F0A775448D#):
  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:Uninitialized
   fun:_ZN7testing8internal11CmpHelperLEIiiEENS_15AssertionResultEPKcS4_RKT_RKT0_
   fun:_ZN6libyuv41libyuvTest_BayerBGGRToARGB_Unaligned_Test8TestBodyEv
}

16:15:10 memcheck_analyze.py [ERROR] Command: src/out/Release/libyuv_unittest
UninitCondition
Conditional jump or move depends on uninitialised value(s)
  testing::AssertionResult testing::internal::CmpHelperLE<int, int>(char const*, char const*, int const&, int const&) (/mnt/data/b/build/slave/linux_memcheck/build/src/out/Release/libyuv_unittest)
  libyuv::libyuvTest_BayerBGGRToARGB_Opt_Test::TestBody() (/mnt/data/b/build/slave/linux_memcheck/build/src/out/Release/libyuv_unittest)
Suppression (error hash=#BF6864F7C79E2777#):
  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:Uninitialized
   fun:_ZN7testing8internal11CmpHelperLEIiiEENS_15AssertionResultEPKcS4_RKT_RKT0_
   fun:_ZN6libyuv35libyuvTest_BayerBGGRToARGB_Opt_Test8TestBodyEv
}

16:15:10 memcheck_analyze.py [ERROR] Command: src/out/Release/libyuv_unittest
UninitCondition
Conditional jump or move depends on uninitialised value(s)
  testing::AssertionResult testing::internal::CmpHelperLE<int, int>(char const*, char const*, int const&, int const&) (/mnt/data/b/build/slave/linux_memcheck/build/src/out/Release/libyuv_unittest)
  libyuv::libyuvTest_I411ToI420_Any_Test::TestBody() (/mnt/data/b/build/slave/linux_memcheck/build/src/out/Release/libyuv_unittest)
Suppression (error hash=#9CD32B63F11F31B1#):
  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:Uninitialized
   fun:_ZN7testing8internal11CmpHelperLEIiiEENS_15AssertionResultEPKcS4_RKT_RKT0_
   fun:_ZN6libyuv30libyuvTest_I411ToI420_Any_Test8TestBodyEv
}

16:15:10 memcheck_analyze.py [ERROR] Command: src/out/Release/libyuv_unittest
UninitCondition
Conditional jump or move depends on uninitialised value(s)
  testing::AssertionResult testing::internal::CmpHelperLE<int, int>(char const*, char const*, int const&, int const&) (/mnt/data/b/build/slave/linux_memcheck/build/src/out/Release/libyuv_unittest)
  libyuv::libyuvTest_BayerBGGRToARGB_Invert_Test::TestBody() (/mnt/data/b/build/slave/linux_memcheck/build/src/out/Release/libyuv_unittest)
Suppression (error hash=#E7DE7086949E73A9#):
  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:Uninitialized
   fun:_ZN7testing8internal11CmpHelperLEIiiEENS_15AssertionResultEPKcS4_RKT_RKT0_
   fun:_ZN6libyuv38libyuvTest_BayerBGGRToARGB_Invert_Test8TestBodyEv
}

...
@@@STEP_LOG_LINE@FD701B474254D5FA@For more info on using suppressions see 
http://dev.chromium.org/developers/tree-sheriffs/sheriff-details-chromium/memory
-sheriff#TOC-Suppressing-memory-reports@@@
@@@STEP_LOG_LINE@FD701B474254D5FA@{@@@
@@@STEP_LOG_LINE@FD701B474254D5FA@<insert_a_suppression_name_here>@@@
@@@STEP_LOG_LINE@FD701B474254D5FA@Memcheck:Unaddressable@@@
@@@STEP_LOG_LINE@FD701B474254D5FA@fun:BayerRowBG@@@
@@@STEP_LOG_LINE@FD701B474254D5FA@fun:BayerToARGB@@@
@@@STEP_LOG_LINE@FD701B474254D5FA@fun:BayerBGGRToARGB@@@
@@@STEP_LOG_LINE@FD701B474254D5FA@fun:_ZN6libyuv41libyuvTest_BayerBGGRToARGB_Una
ligned_Test8TestBodyEv@@@
@@@STEP_LOG_LINE@FD701B474254D5FA@}@@@
@@@STEP_LOG_END@FD701B474254D5FA@@@
@@@STEP_FAILURE@@@
@@@STEP_TEXT@memory test: libyuv_unittest@@@
@@@STEP_TEXT@1 disabled@@@
@@@STEP_TEXT@crashed or hung@@@
 killed dbus-daemon with PID 3618
 cleared DBUS_SESSION_BUS_ADDRESS environment variable
program finished with exit code 255
elapsedTime=184.865802

Original comment by fbarch...@chromium.org on 3 Dec 2013 at 4:21

GoogleCodeExporter commented 9 years ago
Fixed in r888

Original comment by fbarch...@chromium.org on 3 Dec 2013 at 12:21