BasicRateTargeting3TemporalLayersFrameDropping causes valgrind warnings

[GoogleCodeExporter]

v1.3.0-1928-g8e9c9f1

not picked up by the nightly valgrind run as it's categorized under 'Large'; 
there's no valgrind run targeting 'Large' tests yet. Full log attached.

$ valgrind ./test_libvpx 
--gtest_filter=VP9/DatarateTestVP9Large.BasicRateTargeting3TemporalLayersFrameDr
opping/9

---

==20120== Memcheck, a memory error detector
==20120== Copyright (C) 2002-2011, and GNU GPL'd, by Julian Seward et al.
==20120== Using Valgrind-3.7.0 and LibVEX; rerun with -h for copyright info
==20120== Command: ./test_libvpx 
--gtest_filter=VP9/DatarateTestVP9Large.BasicRateTargeting3TemporalLayersFrameDr
opping/9
==20120== 
Note: Google Test filter = 
VP9/DatarateTestVP9Large.BasicRateTargeting3TemporalLayersFrameDropping/9-:AVX/*
:AVX2/*
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from VP9/DatarateTestVP9Large
[ RUN      ] 
VP9/DatarateTestVP9Large.BasicRateTargeting3TemporalLayersFrameDropping/9
==20120== Conditional jump or move depends on uninitialised value(s)
==20120==    at 0x6A1F13: nonrd_use_partition (vp9_encodeframe.c:2764)
==20120==    by 0x6A616A: encode_frame_internal (vp9_encodeframe.c:2834)
==20120==    by 0x5CF6A8: encode_frame_to_data_rate (vp9_onyx_if.c:2680)
==20120==    by 0x5D17ED: vp9_get_compressed_data (vp9_onyx_if.c:3254)
==20120==    by 0x5AF224: vp9e_encode (vp9_cx_iface.c:759)
==20120==    by 0x53CF84: vpx_codec_encode (vpx_encoder.c:227)
==20120==    by 0x47AB97: 
libvpx_test::Encoder::EncodeFrameInternal(libvpx_test::VideoSource const&, 
unsigned long) (encode_test_driver.cc:62)
==20120==    by 0x47C5EB: 
libvpx_test::EncoderTest::RunLoop(libvpx_test::VideoSource*) 
(encode_test_driver.cc:22)
==20120==    by 0x45E70C: (anonymous 
namespace)::DatarateTestVP9Large_BasicRateTargeting3TemporalLayersFrameDropping_
Test::TestBody() (datarate_test.cc:564)
==20120==    by 0x6E5E4C: void 
testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, 
void>(testing::Test*, void (testing::Test::*)(), char const*) 
(gtest-all.cc:3562)
==20120==    by 0x6DD8DF: testing::Test::Run() (gtest-all.cc:3635)
==20120==    by 0x6DDBA7: testing::TestInfo::Run() (gtest-all.cc:3810)
==20120== 
==20120== Conditional jump or move depends on uninitialised value(s)
==20120==    at 0x6A1F27: nonrd_use_partition (vp9_encodeframe.c:2764)
==20120==    by 0x6A616A: encode_frame_internal (vp9_encodeframe.c:2834)
==20120==    by 0x5CF6A8: encode_frame_to_data_rate (vp9_onyx_if.c:2680)
==20120==    by 0x5D17ED: vp9_get_compressed_data (vp9_onyx_if.c:3254)
==20120==    by 0x5AF224: vp9e_encode (vp9_cx_iface.c:759)
==20120==    by 0x53CF84: vpx_codec_encode (vpx_encoder.c:227)
==20120==    by 0x47AB97: 
libvpx_test::Encoder::EncodeFrameInternal(libvpx_test::VideoSource const&, 
unsigned long) (encode_test_driver.cc:62)
==20120==    by 0x47C5EB: 
libvpx_test::EncoderTest::RunLoop(libvpx_test::VideoSource*) 
(encode_test_driver.cc:22)
==20120==    by 0x45E70C: (anonymous 
namespace)::DatarateTestVP9Large_BasicRateTargeting3TemporalLayersFrameDropping_
Test::TestBody() (datarate_test.cc:564)
==20120==    by 0x6E5E4C: void 
testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, 
void>(testing::Test*, void (testing::Test::*)(), char const*) 
(gtest-all.cc:3562)
==20120==    by 0x6DD8DF: testing::Test::Run() (gtest-all.cc:3635)
==20120==    by 0x6DDBA7: testing::TestInfo::Run() (gtest-all.cc:3810)
==20120== 
...

Original issue reported on code.google.com by jz...@google.com on 21 Mar 2014 at 2:03

Attachments:

• log.txt

[GoogleCodeExporter]

this seems to fix the warnings: https://gerrit.chromium.org/gerrit/#/c/69304/

Original comment by ya...@chromium.org on 21 Mar 2014 at 3:04

[GoogleCodeExporter]

Original comment by renganat...@google.com on 3 Apr 2014 at 10:15

• Changed state: Fixed