The encoder was leaving uninitialized data in the padding of slices, while the specs seem to mandate the use of 0xFF. This is also the case for the slice prefix, but it seems completely unused.
To validate this, classical vsynth encoding/decoding fate tests for all supported chroma formats are added. Suggestions for being even more concise in the target/rules are welcome. Christophe Gisquet (2): vc2enc: prevent random data vc2: fate tests libavcodec/vc2enc.c | 4 ++++ tests/fate/vcodec.mak | 17 ++++++++++++++++- tests/ref/vsynth/vsynth1-vc2-420p | 4 ++++ tests/ref/vsynth/vsynth1-vc2-420p10 | 4 ++++ tests/ref/vsynth/vsynth1-vc2-420p12 | 4 ++++ tests/ref/vsynth/vsynth1-vc2-422p | 4 ++++ tests/ref/vsynth/vsynth1-vc2-422p10 | 4 ++++ tests/ref/vsynth/vsynth1-vc2-422p12 | 4 ++++ tests/ref/vsynth/vsynth1-vc2-444p | 4 ++++ tests/ref/vsynth/vsynth1-vc2-444p10 | 4 ++++ tests/ref/vsynth/vsynth1-vc2-444p12 | 4 ++++ tests/ref/vsynth/vsynth2-vc2-420p | 4 ++++ tests/ref/vsynth/vsynth2-vc2-420p10 | 4 ++++ tests/ref/vsynth/vsynth2-vc2-420p12 | 4 ++++ tests/ref/vsynth/vsynth2-vc2-422p | 4 ++++ tests/ref/vsynth/vsynth2-vc2-422p10 | 4 ++++ tests/ref/vsynth/vsynth2-vc2-422p12 | 4 ++++ tests/ref/vsynth/vsynth2-vc2-444p | 4 ++++ tests/ref/vsynth/vsynth2-vc2-444p10 | 4 ++++ tests/ref/vsynth/vsynth2-vc2-444p12 | 4 ++++ tests/ref/vsynth/vsynth_lena-vc2-420p | 4 ++++ tests/ref/vsynth/vsynth_lena-vc2-420p10 | 4 ++++ tests/ref/vsynth/vsynth_lena-vc2-420p12 | 4 ++++ tests/ref/vsynth/vsynth_lena-vc2-422p | 4 ++++ tests/ref/vsynth/vsynth_lena-vc2-422p10 | 4 ++++ tests/ref/vsynth/vsynth_lena-vc2-422p12 | 4 ++++ tests/ref/vsynth/vsynth_lena-vc2-444p | 4 ++++ tests/ref/vsynth/vsynth_lena-vc2-444p10 | 4 ++++ tests/ref/vsynth/vsynth_lena-vc2-444p12 | 4 ++++ 29 files changed, 128 insertions(+), 1 deletion(-) create mode 100644 tests/ref/vsynth/vsynth1-vc2-420p create mode 100644 tests/ref/vsynth/vsynth1-vc2-420p10 create mode 100644 tests/ref/vsynth/vsynth1-vc2-420p12 create mode 100644 tests/ref/vsynth/vsynth1-vc2-422p create mode 100644 tests/ref/vsynth/vsynth1-vc2-422p10 create mode 100644 tests/ref/vsynth/vsynth1-vc2-422p12 create mode 100644 tests/ref/vsynth/vsynth1-vc2-444p create mode 100644 tests/ref/vsynth/vsynth1-vc2-444p10 create mode 100644 tests/ref/vsynth/vsynth1-vc2-444p12 create mode 100644 tests/ref/vsynth/vsynth2-vc2-420p create mode 100644 tests/ref/vsynth/vsynth2-vc2-420p10 create mode 100644 tests/ref/vsynth/vsynth2-vc2-420p12 create mode 100644 tests/ref/vsynth/vsynth2-vc2-422p create mode 100644 tests/ref/vsynth/vsynth2-vc2-422p10 create mode 100644 tests/ref/vsynth/vsynth2-vc2-422p12 create mode 100644 tests/ref/vsynth/vsynth2-vc2-444p create mode 100644 tests/ref/vsynth/vsynth2-vc2-444p10 create mode 100644 tests/ref/vsynth/vsynth2-vc2-444p12 create mode 100644 tests/ref/vsynth/vsynth_lena-vc2-420p create mode 100644 tests/ref/vsynth/vsynth_lena-vc2-420p10 create mode 100644 tests/ref/vsynth/vsynth_lena-vc2-420p12 create mode 100644 tests/ref/vsynth/vsynth_lena-vc2-422p create mode 100644 tests/ref/vsynth/vsynth_lena-vc2-422p10 create mode 100644 tests/ref/vsynth/vsynth_lena-vc2-422p12 create mode 100644 tests/ref/vsynth/vsynth_lena-vc2-444p create mode 100644 tests/ref/vsynth/vsynth_lena-vc2-444p10 create mode 100644 tests/ref/vsynth/vsynth_lena-vc2-444p12 -- 2.8.1 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel