vp9 C loop filter produces valgrind warnings

[GoogleCodeExporter]

v1.3.0-328-gc263418

$ ./configure --enable-debug --disable-optimizations --disable-sse4_1 
--disable-ssse3 --disable-sse3 --disable-sse2 --disable-sse --disable-mmx 
--disable-avx

$ valgrind ./test_libvpx --gtest_filter=VP9/TestVectorTest.MD5Match/74
==8801== Memcheck, a memory error detector
==8801== Copyright (C) 2002-2011, and GNU GPL'd, by Julian Seward et al.
==8801== Using Valgrind-3.7.0 and LibVEX; rerun with -h for copyright info
==8801== Command: ./test_libvpx --gtest_filter=VP9/TestVectorTest.MD5Match/74
==8801== 
Note: Google Test filter = VP9/TestVectorTest.MD5Match/74:-AVX/*:-AVX2/*
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from VP9/TestVectorTest
[ RUN      ] VP9/TestVectorTest.MD5Match/74
==8801== Conditional jump or move depends on uninitialised value(s)
==8801==    at 0x5EC825: vp9_loop_filter_horizontal_edge_c 
(vp9_loopfilter_filters.c:27)
==8801==    by 0x5E9257: filter_selectively_horiz (vp9_loopfilter.c:508)
==8801==    by 0x5EB1EF: filter_block_plane (vp9_loopfilter.c:1240)
==8801==    by 0x5EB3B2: vp9_loop_filter_rows (vp9_loopfilter.c:1286)
==8801==    by 0x5EB551: vp9_loop_filter_worker (vp9_loopfilter.c:1319)
==8801==    by 0x59EB02: vp9_worker_execute (vp9_thread.c:221)
==8801==    by 0x63DB29: decode_tile (vp9_decodeframe.c:813)
==8801==    by 0x63DFC4: decode_tiles (vp9_decodeframe.c:909)
==8801==    by 0x63FD6E: vp9_decode_frame (vp9_decodeframe.c:1384)
==8801==    by 0x5A086E: vp9_receive_compressed_data (vp9_onyxd_if.c:348)
==8801==    by 0x59D55C: decode_one (vp9_dx_iface.c:349)
==8801==    by 0x59D8BB: vp9_decode (vp9_dx_iface.c:443)
==8801== 
==8801== Conditional jump or move depends on uninitialised value(s)
==8801==    at 0x5EC853: vp9_loop_filter_horizontal_edge_c 
(vp9_loopfilter_filters.c:28)
==8801==    by 0x5E9257: filter_selectively_horiz (vp9_loopfilter.c:508)
==8801==    by 0x5EB1EF: filter_block_plane (vp9_loopfilter.c:1240)
==8801==    by 0x5EB3B2: vp9_loop_filter_rows (vp9_loopfilter.c:1286)
==8801==    by 0x5EB551: vp9_loop_filter_worker (vp9_loopfilter.c:1319)
==8801==    by 0x59EB02: vp9_worker_execute (vp9_thread.c:221)
==8801==    by 0x63DB29: decode_tile (vp9_decodeframe.c:813)
==8801==    by 0x63DFC4: decode_tiles (vp9_decodeframe.c:909)
==8801==    by 0x63FD6E: vp9_decode_frame (vp9_decodeframe.c:1384)
==8801==    by 0x5A086E: vp9_receive_compressed_data (vp9_onyxd_if.c:348)
==8801==    by 0x59D55C: decode_one (vp9_dx_iface.c:349)
==8801==    by 0x59D8BB: vp9_decode (vp9_dx_iface.c:443)
==8801== 
==8801== Conditional jump or move depends on uninitialised value(s)
==8801==    at 0x5EC881: vp9_loop_filter_horizontal_edge_c 
(vp9_loopfilter_filters.c:29)
==8801==    by 0x5E9257: filter_selectively_horiz (vp9_loopfilter.c:508)
==8801==    by 0x5EB1EF: filter_block_plane (vp9_loopfilter.c:1240)
==8801==    by 0x5EB3B2: vp9_loop_filter_rows (vp9_loopfilter.c:1286)
==8801==    by 0x5EB551: vp9_loop_filter_worker (vp9_loopfilter.c:1319)
==8801==    by 0x59EB02: vp9_worker_execute (vp9_thread.c:221)
==8801==    by 0x63DB29: decode_tile (vp9_decodeframe.c:813)
==8801==    by 0x63DFC4: decode_tiles (vp9_decodeframe.c:909)
==8801==    by 0x63FD6E: vp9_decode_frame (vp9_decodeframe.c:1384)
==8801==    by 0x5A086E: vp9_receive_compressed_data (vp9_onyxd_if.c:348)
==8801==    by 0x59D55C: decode_one (vp9_dx_iface.c:349)
==8801==    by 0x59D8BB: vp9_decode (vp9_dx_iface.c:443)
...

Original issue reported on code.google.com by jz...@google.com on 12 Dec 2013 at 8:59

[GoogleCodeExporter]

Bisected to this commit 

[ff2c96be1fe11f76b72e6f15fd23155627c9bbe4] Implenment on demand border 
extension. In place extend the border now. Next commit will totally remove the 
border.

Since Hangyu is still working with the code I assigned the bug to him..

Original comment by jimbankoski@google.com on 16 Dec 2013 at 5:44

[GoogleCodeExporter]

Original comment by ya...@google.com on 9 Jan 2014 at 11:08

• Changed state: Started

[GoogleCodeExporter]

Original comment by fgalli...@google.com on 2 Sep 2014 at 5:46

• Changed state: Fixed