[FFmpeg-devel] [PATCH] fate: add aac test which occasionally crashes (Ticket 1784)

[Claudio Freire]

On Sun, Apr 3, 2016 at 10:41 PM, James Almer <jamrial at gmail.com> wrote:
> On 4/3/2016 3:29 PM, Claudio Freire wrote:
>> On Sun, Apr 3, 2016 at 3:05 PM, Claudio Freire <klaussfreire at gmail.com> wrote:
>>> On Sat, Apr 2, 2016 at 4:00 PM, Michael Niedermayer
>>> <michael at niedermayer.cc> wrote:
>>>> Ideally that should be fixed before its pushed
>>>>
>>>> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
>>>> ---
>>>>  tests/fate/aac.mak |   11 +++++++++++
>>>>  1 file changed, 11 insertions(+)
>>>>
>>>> diff --git a/tests/fate/aac.mak b/tests/fate/aac.mak
>>>> index 324b05d..3d64031 100644
>>>> --- a/tests/fate/aac.mak
>>>> +++ b/tests/fate/aac.mak
>>>> @@ -213,6 +213,17 @@ fate-aac-ltp-encode: CMP_TARGET = 1270
>>>>  fate-aac-ltp-encode: SIZE_TOLERANCE = 3560
>>>>  fate-aac-ltp-encode: FUZZ = 17
>>>>
>>>> +#Ticket1784
>>>> +FATE_AAC_ENCODE += fate-aac-yoraw-encode
>>>> +fate-aac-yoraw-encode: CMD = enc_dec_pcm adts wav s16le $(TARGET_SAMPLES)/audio-reference/yo.raw-short.wav -c:a aac -fflags +bitexact -flags +bitexact
>>>> +fate-aac-yoraw-encode: CMP = stddev
>>>> +fate-aac-yoraw-encode: REF = $(SAMPLES)/audio-reference/yo.raw-short.wav
>>>> +fate-aac-yoraw-encode: CMP_SHIFT = -12288
>>>> +fate-aac-yoraw-encode: CMP_TARGET = 259
>>>> +fate-aac-yoraw-encode: SIZE_TOLERANCE = 3560
>>>> +fate-aac-yoraw-encode: FUZZ = 17
>>>> +
>>>> +
>>>>  FATE_AAC_ENCODE += fate-aac-pred-encode
>>>>  fate-aac-pred-encode: CMD = enc_dec_pcm adts wav s16le $(TARGET_SAMPLES)/audio-reference/luckynight_2ch_44kHz_s16.wav -profile:a aac_main -c:a aac -aac_is 0 -aac_pns 0 -aac_ms 0 -aac_tns 0 -b:a 128k -cutoff 22050
>>>>  fate-aac-pred-encode: CMP = stddev
>>>
>>>
>>> Fixed now, feel free to push this
>>
>>
>> Never mind, pushed it myself, kept your signoff.
>
> http://fate.ffmpeg.org/report.cgi?time=20160403222945&slot=x86_64-archlinux-gcc-valgrindundef
>
> No memleaks, but a handful of "Conditional jump or move depends on uninitialised value(s)" errors.


Strange. This shouldn't be new or related to this commit.

I'll take a look nonetheless.