summaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* avcodec/vp56: Implement very basic error concealmentMichael Niedermayer2017-05-141-1/+1
| | | | | | | | | This should fix the fate failure due to a truncated last frame. Alternatively the frame could be dropped. Signed-off-by: Michael Niedermayer <[email protected]> (cherry picked from commit d34bf886e963445350c4987f7a9ed77bd9c9a5c7) Signed-off-by: Michael Niedermayer <[email protected]>
* avcodec/vp56: Fix sign typoMichael Niedermayer2017-05-141-1/+1
| | | | | | | | | | | Fixes: 664/clusterfuzz-testcase-4917047475568640 The change to fate is due to a truncated last frames which is now detected as damaged. Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg Signed-off-by: Michael Niedermayer <[email protected]> (cherry picked from commit 513a3494396d0a20233273b3cadcb5ee86485d5c) Signed-off-by: Michael Niedermayer <[email protected]>
* avcodec/dnxhd_parser: take into account compressed frame size and skip itPaul B Mahol2017-03-241-2/+2
| | | | | | | Fixes #6214 and vsynth1-dnxhd-720p-hr-lb. Signed-off-by: Paul B Mahol <[email protected]> (cherry picked from commit e1940d2458353943e2fab6bdb87d2278077e22a5)
* avformat/apngenc: use the stream parameters extradata if availableJames Almer2016-11-182-3/+6
| | | | | | | Fixes remuxing apng streams coming from the apng demuxer, which sends extradata during init. Signed-off-by: James Almer <[email protected]>
* Revert "apngdec: use side data to pass extradata to the decoder"James Almer2016-11-182-6/+3
| | | | | | | | | | | | | | This reverts commit e0c6b32046f4bab7d34be77dd2f03b2a80c86d39. Said commit changed the behavior of the demuxer and decoder in a non backwards compatible way. Demuxers should make extradata available at init if possible, and send new extradata as side data within a packet if needed. A better fix for the remuxing crash will follow. Signed-off-by: James Almer <[email protected]> (cherry picked from commit 16c429166ddf1736972b6ccce84bd3509ec16a34)
* fate: add streamcopy test for apngAndreas Cadhalpun2016-11-172-0/+6
| | | | | | | Reviewed-by: James Almer <[email protected]> Signed-off-by: Andreas Cadhalpun <[email protected]> (cherry picked from commit 719c15aa9ad6983200b78e5dbc17443f649c8af9) Signed-off-by: Andreas Cadhalpun <[email protected]>
* fate: add apng encoding/muxing testAndreas Cadhalpun2016-11-173-0/+16
| | | | | | | | | Also test the fallback to png creation for a single frame. Reviewed-by: James Almer <[email protected]> Signed-off-by: Andreas Cadhalpun <[email protected]> (cherry picked from commit 97792e85c338d129342f5812e2a52048373e57d6) Signed-off-by: Andreas Cadhalpun <[email protected]>
* avformat/matroskaenc: fix cue relative position values when CRC32 is enabledJames Almer2016-10-262-3/+3
| | | | | | | | | | | | | The dynamic buffer does not contain the CRC32 element so calls to avio_tell() don't take it into account. This resulted in CueRelativePosition values being six bytes short. This is a regression since 6724525a1576ca334d2ffdc085620bb44aea7394 Instead of adding yet another custom check for CRC32 to fix a size or an offset, remove the existing ones and reserve the six bytes in the dynamic buffer. Signed-off-by: James Almer <[email protected]> (cherry picked from commit eabbc64728c2fdb74f565aededec2ab023d20699)
* lavf/flvdec: init AVPacket::pos to FLVTAG offsetSuman-2016-10-265-104/+104
| | | | | | Current code doesn't initialize AVPacket::pos. Made it point to FLVTAG so flv_read_packet can decode from pos Signed-off-by: Michael Niedermayer <[email protected]>
* fate: Add MXF D10/DNXHD/DV25 probe testsTobias Rapp2016-10-264-1/+461
| | | | | Signed-off-by: Tobias Rapp <[email protected]> Signed-off-by: Michael Niedermayer <[email protected]>
* lavf: add AV_DISPOSITION_TIMED_THUMBNAILSRodger Combs2016-10-2412-20/+32
| | | | Reviewed-By: Michael Niedermayer <[email protected]>
* fate/hevc: add automatic bsf testRodger Combs2016-10-241-0/+11
|
* fate/aac: add automatic bsf testRodger Combs2016-10-241-2/+8
|
* fate/h264: make mp4toannexb test use auto-BSFRodger Combs2016-10-241-2/+3
|
* ffprobe: report field order for video streamsRodger Combs2016-10-2410-9/+15
|
* avformat/matroskaenc: write DisplayWidth and DisplayHeight elements only if ↵James Almer2016-10-221-1/+1
| | | | | | they differ from PixelWidth and PixelHeight Signed-off-by: James Almer <[email protected]>
* tests/fate: add mxf metadata streams testMark Reid2016-10-213-0/+32
| | | | Signed-off-by: Michael Niedermayer <[email protected]>
* FATE: Add test for muxing discontinuous MP4 fragments with large timestampsDerek Buitenhuis2016-10-191-0/+5
| | | | Signed-off-by: Derek Buitenhuis <[email protected]>
* fate: add swr-convertaudio testMuhammad Faiz2016-10-181-0/+8
| | | | | | | test for flt to s16 should pass on correct rounding to nearest Signed-off-by: Muhammad Faiz <[email protected]>
* avformat/matroskaenc: write a DisplayUnit element when aspect ratio is unknownJames Almer2016-10-153-29/+29
| | | | | | | | | | | We don't currently support values 1 (centimeters), 2 (inches) or 3 (DAR), only the default value 0 (pixels) which doesn't need to be written. The fate refs are updated as unknown SAR is now signaled in the output files with the addition of the new element. Reviewed-by: Carl Eugen Hoyos <[email protected]> Signed-off-by: James Almer <[email protected]>
* avutil: Improved selftest coverage for libavutil/fifo.cThomas Turner2016-10-141-0/+43
| | | | | | Tested functions: av_fifo_generic_peek(), av_fifo_grow() Signed-off-by: Michael Niedermayer <[email protected]>
* fate: add test for firequalizer filterMuhammad Faiz2016-10-142-0/+36
| | | | Signed-off-by: Muhammad Faiz <[email protected]>
* avformat/matroskaenc: don't write a FlagInterlaced element if it would write ↵James Almer2016-10-111-1/+1
| | | | | | | | | | | the default value The spec says: "Mandatory elements with a default value may be left out of the file. In the absence of a mandatory element, the element's default value is used." Reviewed-by: Hendrik Leppkes <[email protected]> Signed-off-by: James Almer <[email protected]>
* avformat/matroskaenc: fix targets for attachment tagsJames Almer2016-10-101-2/+2
| | | | | | | | | Attachment tags were being written targeting non-existent streams in the output file. Also filter filename and mimetype entries, as they are standard elements in the Attachment master. Signed-off-by: James Almer <[email protected]>
* avformat/movenc: Switch mov_write_uuidprof_tag() to avg_frame_rateMichael Niedermayer2016-10-101-1/+1
| | | | | | | Using the stream timebase simply overflows Fix integer overflow in psp framerate computation Signed-off-by: Michael Niedermayer <[email protected]>
* Merge commit 'dc7501e524dc3270335749302c7aa449973625f3'Hendrik Leppkes2016-10-071-0/+2
|\ | | | | | | | | | | | | * commit 'dc7501e524dc3270335749302c7aa449973625f3': checkasm: Issue emms after benchmarking functions Merged-by: Hendrik Leppkes <[email protected]>
| * checkasm: Issue emms after benchmarking functionsMartin Storsjö2016-06-211-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The functions may not clean up properly after using MMX registers. For the normal testing calls, the checkasm_checked_call functions will do the cleanup (and check that functions that should clean up do it as well), but when benchmarking functions that don't clean up, we don't currently properly clean up at all. This causes issues if a benchmarked function is followed by testing of a function that is supposed to not clobber the MMX/FPU state but doesn't touch it at all. Signed-off-by: Martin Storsjö <[email protected]>
* | Merge commit 'beb62dac629603eb074a44c44389c230b5caac7c'Hendrik Leppkes2016-10-072-6/+6
|\| | | | | | | | | | | | | * commit 'beb62dac629603eb074a44c44389c230b5caac7c': Use AVFrame.pts instead of deprecated pkt_pts. Merged-by: Hendrik Leppkes <[email protected]>
* | tests: add -nostdin flag when calling ffmpegRodger Combs2016-10-062-6/+6
| | | | | | | | | | | | This fixes a long-standing issue where running FATE in parallel could result in the terminal being left misconfigured, particularly if a test failed or was canceled wtih ^C.
* | avformat/matroskaenc: write a CRC32 element on InfoJames Almer2016-10-064-30/+30
| | | | | | | | | | | | | | | | | | Finishes implementing ticket #4347 Tested-by: Dave Rice <[email protected]> Tested-by: Jerome Martinez <[email protected]> Reviewed-by: Michael Niedermayer <[email protected]> Signed-off-by: James Almer <[email protected]>
* | avformat/matroskaenc: write a CRC32 element on TagsJames Almer2016-10-064-30/+30
| | | | | | | | | | | | | | | | | | Implements part of ticket #4347 Tested-by: Dave Rice <[email protected]> Tested-by: Jerome Martinez <[email protected]> Reviewed-by: Michael Niedermayer <[email protected]> Signed-off-by: James Almer <[email protected]>
* | avformat/matroskaenc: write a CRC32 element on AttachmentsJames Almer2016-10-061-2/+2
| | | | | | | | | | | | | | | | | | Implements part of ticket #4347 Tested-by: Dave Rice <[email protected]> Tested-by: Jerome Martinez <[email protected]> Reviewed-by: Michael Niedermayer <[email protected]> Signed-off-by: James Almer <[email protected]>
* | avformat/matroskaenc: write a CRC32 element on TracksJames Almer2016-10-064-30/+30
| | | | | | | | | | | | | | | | | | Implements part of ticket #4347 Tested-by: Dave Rice <[email protected]> Tested-by: Jerome Martinez <[email protected]> Reviewed-by: Michael Niedermayer <[email protected]> Signed-off-by: James Almer <[email protected]>
* | avformat/matroskaenc: write a CRC32 element on CuesJames Almer2016-10-062-6/+6
| | | | | | | | | | | | | | | | | | Implements part of ticket #4347 Tested-by: Dave Rice <[email protected]> Tested-by: Jerome Martinez <[email protected]> Reviewed-by: Michael Niedermayer <[email protected]> Signed-off-by: James Almer <[email protected]>
* | avformat/matroskaenc: write a CRC32 element on SeekHeadJames Almer2016-10-067-34/+34
| | | | | | | | | | | | | | | | | | Implements part of ticket #4347 Tested-by: Dave Rice <[email protected]> Tested-by: Jerome Martinez <[email protected]> Reviewed-by: Michael Niedermayer <[email protected]> Signed-off-by: James Almer <[email protected]>
* | avformat/matroskaenc: write a CRC32 element on each ClusterJames Almer2016-10-064-30/+30
| | | | | | | | | | | | | | | | | | Implements part of ticket #4347 Tested-by: Dave Rice <[email protected]> Tested-by: Jerome Martinez <[email protected]> Reviewed-by: Michael Niedermayer <[email protected]> Signed-off-by: James Almer <[email protected]>
* | avformat/matroskaenc: don't reserve space for stream duration tags if the ↵James Almer2016-10-043-4/+4
| | | | | | | | | | | | | | | | | | output is not seekable The durations are never written in that situation. Reviewed-by: Michael Niedermayer <[email protected]> Signed-off-by: James Almer <[email protected]>
* | fate: add bitexact decode flag to fate-svq3-watermarkJames Almer2016-10-031-1/+1
| | | | | | | | | | | | Fixes failure with mmxext or 3dnow cpuflags enabled but ssse3 disabled Signed-off-by: James Almer <[email protected]>
* | fate: Add PSP copy testMichael Niedermayer2016-10-032-0/+292
| | | | | | | | Signed-off-by: Michael Niedermayer <[email protected]>
* | ffmpeg: use new decode APIwm42016-10-013-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a bit messy, mainly due to timestamp handling. decode_video() relied on the fact that it could set dts on a flush/drain packet. This is not possible with the old API, and won't be. (I think doing this was very questionable with the old API. Flush packets should not contain any information; they just cause a FIFO to be emptied.) This is replaced with checking the best_effort_timestamp for AV_NOPTS_VALUE, and using the suggested DTS in the drain case. The modified tests (fate-cavs and others) still fails due to dropping the last frame. This happens because the timestamp of the last frame goes backwards (ffprobe -show_frames shows the same thing). I suspect that this "worked" due to the best effort timestamp logic picking the DTS over the decreasing PTS. Since this logic is in libavcodec (where it probably shouldn't be), this can't be easily fixed. The timestamps of the cavs samples are weird anyway, so I chose not to fix it. Another strange thing is the timestamp handling in the video path of process_input_packet (after the decode_video() call). It looks like the code to increase next_dts and next_pts should be run every time a frame is decoded - but it's needed even if output is skipped.
* | fate: Add regression test for Ticket 186Michael Niedermayer2016-10-012-1/+359
| | | | | | | | Signed-off-by: Michael Niedermayer <[email protected]>
* | fate: Add fate-svq3-watermarkMichael Niedermayer2016-10-012-1/+17
| | | | | | | | Signed-off-by: Michael Niedermayer <[email protected]>
* | lavf/mov: Read display aspect ratio from ares atom also for dnxhd.Carl Eugen Hoyos2016-09-291-1/+1
| | | | | | | | | | Fixes aspect ratio of sample in ticket #2125. Fixes aspect ratio of sample in ticket #5325.
* | lavf/mov.c: Make audio timestamps strictly monotonically increasing inside ↵Sasi Inguva2016-09-292-2/+2
| | | | | | | | | | | | | | | | | | an edit list. Fixes gapless decoding. Adjust skip_samples field correctly in case of DISCARDed audio frames. Signed-off-by: Sasi Inguva <[email protected]> Signed-off-by: Michael Niedermayer <[email protected]>
* | ffmpeg: switch to the new BSF APIClément Bœsch2016-09-244-59/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit is initially largely based on commit 4426540 from Anton Khirnov <[email protected]> and two following fixes (80fb19b and fe7b21c) which were previously skipped respectively in 98e3153, c9ee36e, and 7fe7cdc. mpeg4-bsf-unpack-bframes FATE reference is updated because the bsf filter now actually fixes the extradata (mpeg4_unpack_bframes_init() changing one byte is now honored on the output extradata). The FATE references for remove_extra change because the packet flags were wrong and the keyframes weren't marked, causing the bsf relying on these proprieties to not actually work as intended. The following was fixed by James Almer: The filter option arguments are now also parsed correctly. A hack to propagate extradata changed by bitstream filters after the first av_bsf_receive_packet() call is added to maintain the current behavior. This was previously done by av_bitstream_filter_filter() and is needed for the aac_adtstoasc bsf. The exit_on_error was not being checked anymore, and led to an exit error in the last frame of h264_mp4toannexb test. Restoring this behaviour prevents erroring out. The test is still changed as a result due to the badly filtered frame now not being written after the failure. Signed-off-by: Clément Bœsch <[email protected]> Signed-off-by: James Almer <[email protected]>
* | ffprobe.c: Indicate decode-but-discard packets when doing -show_packets.Sasi Inguva2016-09-2312-440/+440
| | | | | | | | | | Signed-off-by: Sasi Inguva <[email protected]> Signed-off-by: Michael Niedermayer <[email protected]>
* | fate: add Test for h264_mp4toannexb (ticket2991)Michael Niedermayer2016-09-232-0/+132
| |
* | avcodec/remove_extradata_bsf: Fix AVoption parameter max valueJames Almer2016-09-231-28/+27
| |
* | tests/fate/ffmpeg: Remove dead automatic remove extradata test update the ↵Michael Niedermayer2016-09-232-38/+2
| | | | | | | | | | | | | | keyframe test Found-by: jamrial Signed-off-by: Michael Niedermayer <[email protected]>
* | tests/fate: Add fate-ffmpeg-bsf-remove-* testsMichael Niedermayer2016-09-235-0/+145
| | | | | | | | Signed-off-by: Michael Niedermayer <[email protected]>