diff options
author | Alexander Voronov <avoronov@graphics.cs.msu.ru> | 2014-07-14 16:36:21 +0400 |
---|---|---|
committer | Adrian Grange <agrange@google.com> | 2014-07-15 11:06:15 -0700 |
commit | 0aa2af55b5646ffc37d00714226997df13253534 (patch) | |
tree | 6bfe213a3be0128ca37c77bf921878b462cdca71 /test | |
parent | 6ce515b9ff9000740bc47bb2dfa1b80df6ec2fc9 (diff) | |
download | libvpx-0aa2af55b5646ffc37d00714226997df13253534.tar libvpx-0aa2af55b5646ffc37d00714226997df13253534.tar.gz libvpx-0aa2af55b5646ffc37d00714226997df13253534.tar.bz2 libvpx-0aa2af55b5646ffc37d00714226997df13253534.zip |
Fix show_existing_frame not decreasing frame buffer ref counter.
The issue was introduced by commit g7c43fb6. If current frame
is repeated from existing-ref pool, frame buffer ref counter
is not decreased, so buffer isn't released. Decoder fails being
unable to allocate new frame buffer at some point.
Added a test vector to verify that the condition will not
recur later. Test vector was generated by the code in this patch:
https://gerrit.chromium.org/gerrit/#/c/70862/
Change-Id: I8af96eb5b9670176e01a281d2e18bd458712cf78
Diffstat (limited to 'test')
-rw-r--r-- | test/test-data.sha1 | 2 | ||||
-rw-r--r-- | test/test.mk | 2 | ||||
-rw-r--r-- | test/test_vectors.cc | 2 |
3 files changed, 5 insertions, 1 deletions
diff --git a/test/test-data.sha1 b/test/test-data.sha1 index d5a116558..02b43d9e1 100644 --- a/test/test-data.sha1 +++ b/test/test-data.sha1 @@ -655,6 +655,8 @@ e3ab35d4316c5e81325c50f5236ceca4bc0d35df vp90-2-15-segkey.webm.md5 8f46ba5f785d0c2170591a153e0d0d146a7c8090 vp90-2-15-segkey_adpq.webm.md5 698a6910a97486b833073ef0c0b18d75dce57ee8 vp90-2-16-intra-only.webm 5661b0168752969f055eec37b05fa9fa947dc7eb vp90-2-16-intra-only.webm.md5 +c01bb7938f9a9f25e0c37afdec2f2fb73b6cc7fa vp90-2-17-show-existing-frame.webm +cc75f351818b9a619818f5cc77b9bc013d0c1e11 vp90-2-17-show-existing-frame.webm.md5 0321d507ce62dedc8a51b4e9011f7a19aed9c3dc vp91-2-04-yuv444.webm 367e423dd41fdb49aa028574a2cfec5c2f325c5c vp91-2-04-yuv444.webm.md5 76024eb753cdac6a5e5703aaea189d35c3c30ac7 invalid-vp90-2-00-quantizer-00.webm.ivf.s5861_r01-05_b6-.ivf diff --git a/test/test.mk b/test/test.mk index f624649f0..020175db1 100644 --- a/test/test.mk +++ b/test/test.mk @@ -773,6 +773,8 @@ LIBVPX_TEST_DATA-$(CONFIG_VP9_DECODER) += vp90-2-15-segkey_adpq.webm LIBVPX_TEST_DATA-$(CONFIG_VP9_DECODER) += vp90-2-15-segkey_adpq.webm.md5 LIBVPX_TEST_DATA-$(CONFIG_VP9_DECODER) += vp90-2-16-intra-only.webm LIBVPX_TEST_DATA-$(CONFIG_VP9_DECODER) += vp90-2-16-intra-only.webm.md5 +LIBVPX_TEST_DATA-$(CONFIG_VP9_DECODER) += vp90-2-17-show-existing-frame.webm +LIBVPX_TEST_DATA-$(CONFIG_VP9_DECODER) += vp90-2-17-show-existing-frame.webm.md5 LIBVPX_TEST_DATA-$(CONFIG_VP9_DECODER) += vp91-2-04-yuv444.webm LIBVPX_TEST_DATA-$(CONFIG_VP9_DECODER) += vp91-2-04-yuv444.webm.md5 diff --git a/test/test_vectors.cc b/test/test_vectors.cc index 495588715..4ea4b9dab 100644 --- a/test/test_vectors.cc +++ b/test/test_vectors.cc @@ -180,7 +180,7 @@ const char *const kVP9TestVectors[] = { "vp90-2-14-resize-fp-tiles-8-16.webm", "vp90-2-14-resize-fp-tiles-8-1.webm", "vp90-2-14-resize-fp-tiles-8-2.webm", "vp90-2-14-resize-fp-tiles-8-4.webm", "vp90-2-15-segkey.webm", "vp90-2-15-segkey_adpq.webm", - "vp90-2-16-intra-only.webm", + "vp90-2-16-intra-only.webm", "vp90-2-17-show-existing-frame.webm", "vp91-2-04-yuv444.webm", }; const int kNumVP9TestVectors = NELEMENTS(kVP9TestVectors); |