summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* avformat/matroskaenc: CosmeticsAndreas Rheinhardt2019-10-211-18/+18
| | | | | | | | | | Contains renaming of variables (e.g. mkv_write_cues() contained variables called tracknum that actually contain the index of a track in s->streams and not the track number (which can differ in case an explicit dash track number is set)). Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> Signed-off-by: James Almer <jamrial@gmail.com>
* avformat/matroskaenc: Use more appropriate function nameAndreas Rheinhardt2019-10-211-10/+8
| | | | | | | | | | | | | | mkv_start_new_cluster() actually didn't start a new cluster, but ended the old one instead and emitted a debug message that it had started a new cluster. This has been changed: The debug message has been moved to the place that really starts a new cluster and the function has been renamed to mkv_end_cluster(). Furthermore, without this debug message the function can be used for flushing. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> Signed-off-by: James Almer <jamrial@gmail.com>
* avformat/matroskaenc: Remove redundant assertAndreas Rheinhardt2019-10-211-1/+1
| | | | | | | | | | | | | | The Matroska muxer groups index entries with the same pts together in order to save a few bytes. Because of Matroska's variable-length length fields, mkv_write_cues() does this by first finding out how many index entries will be grouped together before actually writing them. Currently, it is asserted at both of these stages that the stream index of the list of designated index entries is valid. But the second assert is redundant, because the very same index entries have already been checked. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> Signed-off-by: James Almer <jamrial@gmail.com>
* avformat/matroskaenc: Move track-related fields to mkv_trackAndreas Rheinhardt2019-10-211-26/+15
| | | | | Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> Signed-off-by: James Almer <jamrial@gmail.com>
* avformat/matroskaenc: Use ffio_free_dyn_bufAndreas Rheinhardt2019-10-211-17/+6
| | | | | | | instead of replicating its behaviour. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> Signed-off-by: James Almer <jamrial@gmail.com>
* avformat/matroskaenc: Fix memleak upon failureAndreas Rheinhardt2019-10-211-18/+16
| | | | | | | | | | | | | | | | | The Matroska muxer up until now leaked memory in two scenarios: 1. If an error happened during writing the trailer, as mkv_write_trailer() returned early without cleaning up. 2. If mkv_write_header() indicated success despite an error in the underlying AVIOContext. In this case avformat_write_header() returned the IO error and according to the API the caller is not allowed to call av_write_trailer(), so that no cleanup happened for the allocations made in mkv_write_header(). This has been fixed by using a dedicated deinit function. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> Signed-off-by: James Almer <jamrial@gmail.com>
* avfilter/vsrc_mptestsrc: simplify the code and change the type of frameLimin Wang2019-10-211-13/+14
| | | | | Signed-off-by: Limin Wang <lance.lmwang@gmail.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avcodec/atrac9dec: Set channelsMichael Niedermayer2019-10-211-0/+1
| | | | | | | | | Fixes: null pointer dereference Fixes: 18341/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ATRAC9_fuzzer-5681203490848768 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Reviewed-by: Lynne <dev@lynne.ee> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avfilter/vsrc_mptestsrc: add options to set the maximum number of framesLimin Wang2019-10-212-12/+20
| | | | | Signed-off-by: Limin Wang <lance.lmwang@gmail.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avfilter/vf_unsharp: rename config_props -> config_input, link -> inlinkLimin Wang2019-10-211-8/+8
| | | | | Signed-off-by: Limin Wang <lance.lmwang@gmail.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avformat/oggenc: free buffered page lists while uninitializing the muxerJames Almer2019-10-211-0/+9
| | | | | | | If the trailer is never writen, there could be buffered pages that would leak. Reviewed-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> Signed-off-by: James Almer <jamrial@gmail.com>
* avformat/flacenc: add a deinit functionJames Almer2019-10-211-2/+9
| | | | | Reviewed-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> Signed-off-by: James Almer <jamrial@gmail.com>
* avformat/mp3enc: add init and deinit functionsJames Almer2019-10-211-4/+20
| | | | | Reviewed-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> Signed-off-by: James Almer <jamrial@gmail.com>
* avformat/ttaenc: add a deinit functionJames Almer2019-10-211-0/+10
| | | | | Reviewed-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> Signed-off-by: James Almer <jamrial@gmail.com>
* avformat/avienc: add deinit functionJames Almer2019-10-211-5/+15
| | | | | | | Fixes ticket #8302 Reviewed-by: Paul B Mahol <onemda@gmail.com> Signed-off-by: James Almer <jamrial@gmail.com>
* avformat: call AVOutputFormat->deinit() when freeing the contextJames Almer2019-10-212-7/+13
| | | | | | | | | | | | Despite the doxy stating that it's called when the muxer is destroyed, this was not true in practice. It's only called by av_write_trailer() and on init() failure. An AVFormatContext may be closed without writing the trailer if errors ocurred while muxing packets, so in order to prevent memory leaks, it should effectively be called when freeing the muxer. Signed-off-by: James Almer <jamrial@gmail.com>
* x86/vf_transpose: fix cpuflags checkJames Almer2019-10-211-2/+2
| | | | Signed-off-by: James Almer <jamrial@gmail.com>
* avfilter/vf_transpose: add x86 SIMDPaul B Mahol2019-10-215-8/+157
|
* avfilter/x86/vf_atadenoise: fix commentPaul B Mahol2019-10-211-1/+1
|
* avfilter/af_join: fix possible memory leaksPaul B Mahol2019-10-211-1/+4
| | | | | Allocation of input frames is independent from allocation of new input pads.
* avfilter/af_silencedetect: change mono default to integer literalLimin Wang2019-10-211-2/+2
| | | | | Signed-off-by: Limin Wang <lance.lmwang@gmail.com> Reviewed-by: Gyan Doshi <ffmpeg@gyani.pro>
* avfilter/af_silencedetect: use AV_OPT_TYPE_DURATIONLimin Wang2019-10-213-6/+10
| | | | | Reviewed-by: Moritz Barsnick <barsnick@gmx.net> Signed-off-by: Limin Wang <lance.lmwang@gmail.com>
* avfilter: add bilateral filterPaul B Mahol2019-10-216-1/+396
| | | | Signed-off-by: Paul B Mahol <onemda@gmail.com>
* avfilter/vf_nlmeans: round values toward nearest integerPaul B Mahol2019-10-211-1/+1
| | | | | Instead of rounding toward zero and thus producing darker output.
* avfilter/af_silencedetect: document metadataLimin Wang2019-10-212-2/+10
| | | | | Reviewed-by: Moritz Barsnick <barsnick@gmx.net> Signed-off-by: Limin Wang <lance.lmwang@gmail.com>
* avcodec/alsdec: Check that input space for header exists in ↵Michael Niedermayer2019-10-201-0/+3
| | | | | | | | | | read_diff_float_data() Fixes: Timeout (21sec -> 8sec) Fixes: 17832/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ALS_fuzzer-5737092172218368 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avformat/pjsdec: Check duration for overflowMichael Niedermayer2019-10-201-0/+2
| | | | | | | | Fixes: signed integer overflow: -3 - 9223372036854775807 cannot be represented in type 'long' Fixes: 17828/clusterfuzz-testcase-minimized-ffmpeg_DEMUXER_fuzzer-5645915116797952 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avcodec/agm: Check for reference frame earlierMichael Niedermayer2019-10-201-4/+7
| | | | | | | | Fixes: Timeout (14sec - 120ms) Fixes: 17824/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_AGM_fuzzer-5639825457152000 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avcodec/ptx: Check that the input contains at least one lineMichael Niedermayer2019-10-201-0/+3
| | | | | | | | Fixes: Timeout (19sec -> 44ms) Fixes: 17816/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_PTX_fuzzer-5704459950227456 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avcodec/alac: Fix integer overflow in LPCMichael Niedermayer2019-10-201-1/+1
| | | | | | | | Fixes: signed integer overflow: 2147483628 + 128 cannot be represented in type 'int' Fixes: 17783/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ALAC_fuzzer-5146470595952640 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avcodec/smacker: Fix integer overflows in pred[] in smka_decode_frame()Michael Niedermayer2019-10-201-2/+2
| | | | | | | | Fixes: signed integer overflow: -2147481503 + -32732 cannot be represented in type 'int' Fixes: 17782/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_SMACKAUD_fuzzer-5769672225456128 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avcodec/aliaspixdec: Check input size against minimal picture sizeMichael Niedermayer2019-10-201-0/+3
| | | | | | | | Fixes: Timeout (15sec -> 72ms) Fixes: 17774/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ALIAS_PIX_fuzzer-5193929107963904 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avcodec/ffwavesynth: Fix integer overflows in pink noise additionMichael Niedermayer2019-10-201-2/+2
| | | | | | | | Fixes: signed integer overflow: -1795675744 + -1926578528 cannot be represented in type 'int' Fixes: 17741/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_FFWAVESYNTH_fuzzer-5131336402075648 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avcodec/vc1_block: Fixes integer overflow in vc1_decode_i_block_adv()Michael Niedermayer2019-10-201-1/+1
| | | | | | | | Fixes: signed integer overflow: 62220 * 262144 cannot be represented in type 'int' Fixes: 17145/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_VC1IMAGE_fuzzer-5667394743173120 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* tools/target_dec_fuzzer: Adjust threshold for DXVMichael Niedermayer2019-10-201-0/+1
| | | | | | | | Fixes: TImeout (20sec -> 4sec) Fixes: 17735/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_DXV_fuzzer-5723368317255680 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avcodec/wmalosslessdec: Check block_alignMichael Niedermayer2019-10-201-2/+2
| | | | | | | | | Fixes: NULL pointer dereference Fixes: 18331/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_WMALOSSLESS_fuzzer-5652847445671936 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Reviewed-by: Paul B Mahol <onemda@gmail.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avfilter/vf_tile: fix memory leakPaul B Mahol2019-10-201-0/+1
| | | | Fixes #8313
* avfilter/vf_atadenoise: add support for commandsPaul B Mahol2019-10-202-4/+25
|
* avfilter/vf_shuffleframes: improve error messagePaul B Mahol2019-10-201-1/+1
|
* avformat/options: don't call avformat_free_context() within ↵James Almer2019-10-191-4/+6
| | | | | | | | | avformat_alloc_context() avformat_free_context() expects AVFormatContext->internal to not be NULL. Reviewed-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> Signed-off-by: James Almer <jamrial@gmail.com>
* avformat/nutenc: free all missing dynamic AVIOContext on header writing failureJames Almer2019-10-191-4/+7
| | | | | | Fixes part of ticket #8316 Signed-off-by: James Almer <jamrial@gmail.com>
* avfilter/af_tremolo: fix heap-buffer overflowPaul B Mahol2019-10-191-3/+5
| | | | Fixes #8317
* avfilter/vf_fftdnoiz: round toward nearest integerPaul B Mahol2019-10-191-2/+2
| | | | | Instead of rounding toward zero and thus producing darker output.
* avfilter/vf_bm3d: round values toward nearest integerPaul B Mahol2019-10-191-2/+2
| | | | | Instead of rounding toward zero and thus producing darker output.
* avfilter/vf_bm3d: forward status backPaul B Mahol2019-10-191-0/+2
|
* mpeg4_unpack_bframes: Merge close and flushAndreas Rheinhardt2019-10-191-9/+3
| | | | | | | They are identical now. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avcodec/zmbv: Call decode_intra directlyAndreas Rheinhardt2019-10-191-11/+5
| | | | | | | | | | | | | | | zmbv has only one function for decoding intra frames, namely decode_intra; and yet up until now it has been called via a function pointer. This has been changed. This also removes spec-incompliant conversions between function pointers and pointers of type void * and thereby fixes the warning "ISO C forbids assignment between function pointer and ‘void *’" that GCC emits with the -pedantic option. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> Reviewed-by: Paul B Mahol <onemda@gmail.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avcodec/g729dec: Avoid computing invalid temporary pointers for ↵Michael Niedermayer2019-10-191-6/+7
| | | | | | | | | ff_acelp_weighted_vector_sum() Fixes: Ticket8176 Reviewed-by: Paul B Mahol <onemda@gmail.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avcodec/g729postfilter: Fix left shift of negative valueMichael Niedermayer2019-10-191-3/+3
| | | | | | | Fixes: Ticket8176 Reviewed-by: Paul B Mahol <onemda@gmail.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avfilter/vf_fieldhint: add support for duplicating fieldsPaul B Mahol2019-10-192-3/+33
| | | | Fixes #7066
OpenPOWER on IntegriCloud