[FFmpeg-devel] [PATCH] Add FATE test for rare overflow-prone case

Vitor Sessak vitor1001
Sun Jan 23 20:04:41 CET 2011


On 01/23/2011 07:15 PM, M?ns Rullg?rd wrote:
> Vitor Sessak<vitor1001 at gmail.com>  writes:
>
>> $subj, see issue 2547.
>>
>> -Vitor
>>
>> PS: no need to wait boxes to rsync, this sample was uploaded several
>> days ago.
>>
>>  From 7d36f70ea1693233085939c66491354cd82fad9a Mon Sep 17 00:00:00 2001
>> From: Vitor Sessak<vitor1001 at gmail.com>
>> Date: Sun, 23 Jan 2011 18:58:34 +0100
>> Subject: [PATCH] Add FATE test for rare overflow-prone case
>>
>> ---
>>   tests/fate/h264.mak                |    2 ++
>>   tests/ref/fate/h264-might-overflow |   29 +++++++++++++++++++++++++++++
>>   2 files changed, 31 insertions(+), 0 deletions(-)
>>   create mode 100644 tests/ref/fate/h264-might-overflow
>>
>> diff --git a/tests/fate/h264.mak b/tests/fate/h264.mak
>> index 25517fe..5d6fa43 100644
>> --- a/tests/fate/h264.mak
>> +++ b/tests/fate/h264.mak
>> @@ -167,6 +167,7 @@ FATE_H264 = aud_mw_e                                                    \
>>   FATE_H264  := $(FATE_H264:%=fate-h264-conformance-%)                    \
>>                 fate-h264-interlace-crop                                  \
>>                 fate-h264-lossless                                        \
>> +              fate-h264-might-overflow                                  \
>>
>>   FATE_TESTS += $(FATE_H264)
>>   fate-h264: $(FATE_H264)
>> @@ -339,3 +340,4 @@ fate-h264-conformance-sva_nl2_e: CMD = framecrc  -i $(SAMPLES)/h264-conformance/
>>
>>   fate-h264-interlace-crop: CMD = framecrc  -vframes 3 -i $(SAMPLES)/h264/interlaced_crop.mp4
>>   fate-h264-lossless: CMD = framecrc -i $(SAMPLES)/h264/lossless.h264
>> +fate-h264-might-overflow: CMD = framecrc -i $(SAMPLES)/h264/H264_might_overflow.mkv
>
> I'd prefer a slightly more descriptive name for this test.  What
> does it actually test?  "Might overflow" is a lie too, since a correct
> decoder does not overflow.

Of course I meant "might overflow if implemented in a buggy way", but 
new patch attached.

-Vitor



More information about the ffmpeg-devel mailing list