- 12 Apr, 2020 7 commits
-
-
Andreas Rheinhardt authored
According to C99, there has to be at least one argument for every ... in a variadic function-like macro. In practice most (all?) compilers also allow to leave it completely out, but it is nevertheless required: In a variadic macro "there shall be more arguments in the invocation than there are parameters in the macro definition (excluding the ...)." (C99, 6.10.3.4). CBS (not the framework itself, but the macros used in the cbs_*_syntax_template.c files) relies on the compiler allowing to leave a variadic macro argument out. This leads to warnings when compiling in -pedantic mode, e.g. "warning: must specify at least one argument for '...' parameter of variadic macro [-Wgnu-zero-variadic-macro-arguments]" from Clang. Most of these warnings can be easily avoided: The syntax_templates mostly contain helper macros that expand to more complex variadic macros and these helper macros often omit an argument for the .... Modifying them to always expand to complex macros with an empty argument for the ... at the end fixes most of these warnings: The number of warnings went down from 400 to 0 for cbs_av1, from 1114 to 32 for cbs_h2645, from 38 to 0 for cbs_jpeg, from 166 to 0 for cbs_mpeg2 and from 110 to 8 for cbs_vp9. These eight remaining warnings for cbs_vp9 have been fixed by switching to another macro in cbs_vp9_syntax_template: The fixed values for the sync bytes as well as the trailing bits for byte-alignment are now read via the fixed() macro (this also adds a check to ensure that trailing bits are indeed zero as they have to be). Reviewed-by: Mark Thompson <sw@jkqxz.net> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
-
Carl Eugen Hoyos authored
Do not limit the array allocation functions and av_calloc() to allocations of INT_MAX, instead depend on max_alloc_size like av_malloc(). Allows a workaround for ticket #7140.
-
Andreas Rheinhardt authored
Forgotten in ab502fab. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
-
Michael Niedermayer authored
Fixes: Timeout (147sec -> 1sec) Fixes: 20764/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ZEROCODEC_fuzzer-5068274603917312 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpegSigned-off-by: Michael Niedermayer <michael@niedermayer.cc>
-
Michael Niedermayer authored
Fixes: Infinite loop Fixes: Ticket8594 Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-
Michael Niedermayer authored
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-
Rosen Penev authored
On other platforms, the functions are named get_cabac_inline_xxx but not this one. There's also a define. Signed-off-by: Rosen Penev <rosenp@gmail.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-
- 11 Apr, 2020 19 commits
-
-
Peter Ross authored
fix link error introduced in 481ebb1cSigned-off-by: Peter Ross <pross@xvid.org> Reviewed-by: Anton Khirnov <anton@khirnov.net>
-
James Zern authored
tradeof -> trade-off Signed-off-by: James Zern <jzern@google.com>
-
Wang Cao authored
Signed-off-by: Wang Cao <wangcao@google.com> Signed-off-by: James Zern <jzern@google.com>
-
John Stebbins authored
dts would start over at the beginning of each trun when they should be computed contiguously for each trun in a traf Fixes ticket 8070 Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-
John Stebbins authored
If some but not all moof's are referenced in an sidx, whole fragments were being skipped. Fixes tickets 7377, 7389, and 8502 Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-
Michael Niedermayer authored
Fixes: shift exponent 32 is too large for 32-bit type 'int' Fixes: 21200/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_XAN_DPCM_fuzzer-5754704894361600 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpegReviewed-by: Anton Khirnov <anton@khirnov.net> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-
Gautam Ramakrishnan authored
The calculation of precinct boundaries has been fixed. The precinct boundaries were calculated as an offset to the band boundary, but must instead be calculated as an offset from the reslevel. This patch fixes #4669 and #4679. Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-
Gautam Ramakrishnan authored
This patch fixes an error where the COC marker overrides all data of the SPcod field of the COD marker. It must override only one bit of SPcod field. This now allows p0_08.j2k to be decoded correctly (mentioned in #4679). Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-
Andreas Rheinhardt authored
This is required to use it as an AVCodec.supported_samplerates array. Adding the sentinel has been forgotten in 4679a474. Without it e.g. the FATE-test ffmpeg-filter_complex_audio fails with ASAN. Reviewed-by: Carl Eugen Hoyos <ceffmpeg@gmail.com> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
-
Zane van Iperen authored
Signed-off-by: Zane van Iperen <zane@zanevaniperen.com>
-
Carl Eugen Hoyos authored
-
Carl Eugen Hoyos authored
Fixes ticket #7113.
-
Carl Eugen Hoyos authored
Modes > MODE_SID are already filtered out at this point. Simplifies the following change.
-
Rosen Penev authored
Place HAVE_MIPSFPU further up so that functions that use floating point ASM are defined away. Otherwise compilation failures result when soft float in enabled on the toolchain. Signed-off-by: Rosen Penev <rosenp@gmail.com>
-
Carl Eugen Hoyos authored
-
Mattias Wadman authored
Regression since 8d3630c5 where keys were changed to not be touppered but the picture block strcmp was not changed to be case-insensitive. Fixes ticket #8608.
-
John Stebbins authored
This was inadvertantly removed in 4a9d32baReviewed-by: Gyan Doshi <ffmpeg@gyani.pro>
-
Gyan Doshi authored
-
Gyan Doshi authored
Deprecated in 2015 in 8bc708fc
-
- 10 Apr, 2020 14 commits
-
-
John Stebbins authored
Signed-off-by: Philip Langdale <philipl@overt.org>
-
John Stebbins authored
Signed-off-by: Philip Langdale <philipl@overt.org>
-
John Stebbins authored
Signed-off-by: Philip Langdale <philipl@overt.org>
-
John Stebbins authored
Signed-off-by: Philip Langdale <philipl@overt.org>
-
John Stebbins authored
When the buffer is too small, return AVERROR_BUFFER_TOO_SMALL Signed-off-by: Philip Langdale <philipl@overt.org>
-
John Stebbins authored
Signed-off-by: Philip Langdale <philipl@overt.org>
-
John Stebbins authored
encode buffer size was ignored Signed-off-by: Philip Langdale <philipl@overt.org>
-
Philip Langdale authored
-
John Stebbins authored
Signed-off-by: Philip Langdale <philipl@overt.org>
-
John Stebbins authored
If the video dimensions are different than the ASS play_res then the font sizes need to be adjusted to get the same apparent render size. Signed-off-by: Philip Langdale <philipl@overt.org>
-
John Stebbins authored
Initializes the mov text sample description from the ASS header and creates an mov font table from the fonts available in the ASS Styles. Signed-off-by: Philip Langdale <philipl@overt.org>
-
John Stebbins authored
Signed-off-by: Philip Langdale <philipl@overt.org>
-
John Stebbins authored
Signed-off-by: Philip Langdale <philipl@overt.org>
-
John Stebbins authored
Signed-off-by: Philip Langdale <philipl@overt.org>
-