summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* fftools/ffmpeg_sched: remove a triggerable assert in send_to_enc_sq()Anton Khirnov2024-02-241-14/+17
| | | | | | | It can be triggered when send_to_enc_thread() returns AVERROR(ENOMEM). Propagate the error to the caller instead. Reported-by: Andreas Rheinhardt
* fftools/ffmpeg_sched: simplify failure path in sch_dec_send()Anton Khirnov2024-02-241-4/+2
|
* fftools/ffmpeg: declare loop indices inside loopsAnton Khirnov2024-02-246-40/+25
|
* fftools/cmdutils: remove harmful variable shadowingAnton Khirnov2024-02-241-2/+0
| | | | It causes write_option() to return 0 when calling func_arg() fails.
* lavc/vvc_ps: Correct vaule of ALF_NUM_FILTERS_CCFei Wang2024-02-231-1/+1
| | | | Signed-off-by: Fei Wang <[email protected]>
* .mailmap: Update my mailmap entryAndreas Rheinhardt2024-02-231-1/+2
| | | | Signed-off-by: Andreas Rheinhardt <[email protected]>
* avformat/libsrt: Remove manually free AV_OPT_TYPE_STRINGZhao Zhili2024-02-221-3/+0
| | | | Signed-off-by: Zhao Zhili <[email protected]>
* avcodec/audiotoolboxenc: Ensure frame_size isn't zeroZhao Zhili2024-02-221-3/+21
| | | | Signed-off-by: Zhao Zhili <[email protected]>
* avcodec/audiotoolboxenc: remove CAP_VARIABLE_FRAME_SIZE from alacZhao Zhili2024-02-221-1/+1
| | | | | | | | | | | | | | | AV_CODEC_CAP_VARIABLE_FRAME_SIZE has been set for alac_at encoder, which means avctx->frame_size should be zero. However, alac_at encoder also set avctx->frame_size. This leading to assert failure in ffmpeg_sched.c av_assert0(enc->sq_idx[0] >= 0); Actually, the implementation of audiotoolboxenc.c doesn't support frame_size been zero. Fix #10720. Signed-off-by: Zhao Zhili <[email protected]>
* avutil/tx: print debug log at trace levelJames Almer2024-02-221-6/+7
| | | | | | | The output of TX is extremely verbose and makes it harder to find other debug log messages, so print most of it at trace level. Signed-off-by: James Almer <[email protected]>
* avcodec/movtextenc: Don't copy data around unnecessarilyAndreas Rheinhardt2024-02-221-15/+18
| | | | | | | | | Using av_bprint_init_for_buffer() avoids copying data into the internal AVBPrint buffer (or worse: to allocate a temporary buffer in case the internal buffer does not suffice). Signed-off-by: Andreas Rheinhardt <[email protected]>
* avcodec/srtenc, webvttenc: Use av_printf_formatAndreas Rheinhardt2024-02-222-8/+2
| | | | Signed-off-by: Andreas Rheinhardt <[email protected]>
* avcodec/ttmlenc: Remove always-true checkAndreas Rheinhardt2024-02-221-3/+2
| | | | Signed-off-by: Andreas Rheinhardt <[email protected]>
* avcodec/ttmlenc: Don't copy data around unnecessarilyAndreas Rheinhardt2024-02-221-20/+16
| | | | | | | | | Using av_bprint_init_for_buffer() avoids copying data into the internal AVBPrint buffer (or worse: to allocate a temporary buffer in case the internal buffer does not suffice). Signed-off-by: Andreas Rheinhardt <[email protected]>
* avcodec/webvttenc: Don't copy data around unnecessarilyAndreas Rheinhardt2024-02-221-8/+2
| | | | | | | | | | Using av_bprint_init_for_buffer() avoids copying data into the internal AVBPrint buffer (or worse: to allocate a temporary buffer in case the internal buffer does not suffice). It also ensures that the data is always 0-terminated. Signed-off-by: Andreas Rheinhardt <[email protected]>
* avcodec/srtenc: Don't copy data around unnecessarilyAndreas Rheinhardt2024-02-221-8/+2
| | | | | | | | | | | | Using av_bprint_init_for_buffer() avoids copying data into the internal AVBPrint buffer (or worse: to allocate a temporary buffer in case the internal buffer does not suffice). It also ensures that the data is always 0-terminated, whereas the current code never does this and returns success in case the length of the string to write and the size of the buffer coincide. Signed-off-by: Andreas Rheinhardt <[email protected]>
* avcodec/assenc: Use size_t for length of stringAndreas Rheinhardt2024-02-221-2/+2
| | | | Signed-off-by: Andreas Rheinhardt <[email protected]>
* avcodec/cbs_vp8: Don't leave out ... in calls to variadic macrosAndreas Rheinhardt2024-02-221-5/+5
| | | | | | | It is undefined behaviour (see C11, 6.10.3 (4); see also 14dd0a9057019e97ff9438f6cc1502f6922acb85). Signed-off-by: Andreas Rheinhardt <[email protected]>
* avcodec/h264dec: Reindent after the previous commitAndreas Rheinhardt2024-02-221-8/+8
| | | | Signed-off-by: Andreas Rheinhardt <[email protected]>
* avcodec/h264dec: Return early in ff_h264_draw_horiz_band()Andreas Rheinhardt2024-02-221-10/+13
| | | | Signed-off-by: Andreas Rheinhardt <[email protected]>
* configure: Fix a typoMartin Storsjö2024-02-211-1/+1
| | | | Signed-off-by: Martin Storsjö <[email protected]>
* checkasm: Add a "run-checkasm" make targetMartin Storsjö2024-02-211-0/+4
| | | | | | | | | | | | | | | | Contrary to the existing "fate-checkasm", this always prints the tool output, and runs all tests at once instead of splitting it up per target group. This is more useful when the user expects to look directly at the tool output, instead of being part of a full fate run. (On failure with the regular "make fate-checkasm" targets, none of the tool output is printed, but stored in files. If run with reporting set up to the FATE website, the individual failures are uploaded there, but if it is run in some sort of other CI setup, the intermediate files might not be available afterwards for inspection.) Signed-off-by: Martin Storsjö <[email protected]>
* fate/mov: rename a test group to better reflect what it requiresJames Almer2024-02-211-7/+7
| | | | Signed-off-by: James Almer <[email protected]>
* fate/mov: add missing samples dependency to avif/heic testsJames Almer2024-02-211-5/+6
| | | | Signed-off-by: James Almer <[email protected]>
* fate/mov: add missing ffprobe dependency to mov-write-amveJames Almer2024-02-211-2/+4
| | | | Signed-off-by: James Almer <[email protected]>
* avcodec/cbs_vp8: Remove empty flush callbackAndreas Rheinhardt2024-02-211-7/+0
| | | | | | | This callback is optional and should therefore only be set if it actually does something. Signed-off-by: Andreas Rheinhardt <[email protected]>
* fate/image: Enable fate-webp-rgb-lossless-palette-predictor testAndreas Rheinhardt2024-02-211-3/+2
| | | | | | | Postponed in 9f4708c22def8a0f13c3b2bc39baca928bb58aaa because the sample had not been uploaded at that time. Signed-off-by: Andreas Rheinhardt <[email protected]>
* lavc/me_cmp: R-V V ssesunyuechi2024-02-212-0/+77
| | | | | | | | | | | | C908: sse_0_c: 614.7 sse_0_rvv_i32: 138.2 sse_1_c: 302.7 sse_1_rvv_i32: 107.2 sse_2_c: 175.7 sse_2_rvv_i32: 104.2 Signed-off-by: Rémi Denis-Courmont <[email protected]>
* lavc/me_cmp: R-V V pix_abs_y2sunyuechi2024-02-212-0/+56
| | | | | | | | | | C908: pix_abs_0_2_c: 904.0 pix_abs_0_2_rvv_i32: 172.2 pix_abs_1_2_c: 460.0 pix_abs_1_2_rvv_i32: 168.2 Signed-off-by: Rémi Denis-Courmont <[email protected]>
* lavc/me_cmp: R-V V pix_abs_x2sunyuechi2024-02-212-0/+56
| | | | | | | | | | C908: pix_abs_0_1_c: 767.0 pix_abs_0_1_rvv_i32: 196.2 pix_abs_1_1_c: 388.0 pix_abs_1_1_rvv_i32: 185.2 Signed-off-by: Rémi Denis-Courmont <[email protected]>
* lavc/me_cmp: R-V V pix_abssunyuechi2024-02-215-0/+118
| | | | | | | | | | | | | | C908: pix_abs_0_0_c: 534.0 pix_abs_0_0_rvv_i32: 136.2 pix_abs_1_0_c: 287.7 pix_abs_1_0_rvv_i32: 125.2 sad_0_c: 534.0 sad_0_rvv_i32: 136.2 sad_1_c: 287.7 sad_1_rvv_i32: 125.2 Signed-off-by: Rémi Denis-Courmont <[email protected]>
* libswscale/utils: Fix bayer to yuvjMichael Niedermayer2024-02-211-1/+2
| | | | | | | | | Fixes: out of array access. Earlier code assumes that a unscaled bayer to yuvj420 converter exists but the later code then skips yuvj420 Signed-off-by: Michael Niedermayer <[email protected]>
* Revert "swscale: fix sws_setColorspaceDetails after sws_init_context"Michael Niedermayer2024-02-212-4/+8
| | | | | | | | | Suggested by: Niklas Haas in Ticket10824 Fixes: Assertion failure Fixes: Ticket10824 This reverts commit cedf589c09c567b72bf4c1a58db53d94622567e1.
* tools: Add target_sws_fuzzer.cMichael Niedermayer2024-02-213-0/+208
| | | | Signed-off-by: Michael Niedermayer <[email protected]>
* swscale/swscale: Check srcSliceH for bayerMichael Niedermayer2024-02-211-1/+2
| | | | | Fixes: Assertion srcSliceH > 1 failed at libswscale/swscale_unscaled.c:1359 Signed-off-by: Michael Niedermayer <[email protected]>
* swscale/utils: Allocate more dithererrorMichael Niedermayer2024-02-211-1/+1
| | | | | Fixes: out of array read Signed-off-by: Michael Niedermayer <[email protected]>
* Avoid OOM for invalid STCO / CO64 constructions.Dale Curtis2024-02-211-0/+7
| | | | | | | | | | | The `entries` value is read directly from the stream and used to allocate memory. This change clamps `entries` to however many are possible in the remaining atom or file size (whichever is smallest). Fixes https://crbug.com/1429357 Signed-off-by: Dale Curtis <[email protected]> Signed-off-by: Michael Niedermayer <[email protected]>
* fftools/ffmpeg: cosmetics, vertically align structsAnton Khirnov2024-02-214-146/+146
|
* fftools/ffmpeg: move subtitle helpers to ffmpeg_dec, their only userAnton Khirnov2024-02-213-123/+120
|
* fftools/ffmpeg_filter: pass framerate through InputFilterOptionsAnton Khirnov2024-02-213-11/+18
| | | | | | | Rather than read it directly from InputStream. This is a step towards avoiding the assumption that filtergraph inputs are always fed by demuxers.
* fftools/ffmpeg_filter: pass autorotate/reinit flags through InputFilterOptionsAnton Khirnov2024-02-213-6/+17
| | | | | | | Rather than read them directly from InputStream. This is a step towards avoiding the assumption that filtergraph inputs are always fed by demuxers.
* fftools/ffmpeg_filter: pass sub2video canvas size through InputFilterOptionsAnton Khirnov2024-02-213-36/+33
| | | | | | | Rather than read them directly from InputStream. This is a step towards avoiding the assumption that filtergraph inputs are always fed by demuxers.
* fftools/ffmpeg_filter: accept a name from its upstream inputAnton Khirnov2024-02-213-13/+14
| | | | | | | Do not construct the name manually from input file/stream indices. This is a step towards avoiding the assumption that filtergraph inputs are always fed by demuxers.
* fftools/ffmpeg_filter: compute input trim start/end in demuxerAnton Khirnov2024-02-213-26/+32
| | | | | | The computation is based on demuxer properties, so that is the more appropriate place for it. Filter code just receives the desired start time/duration.
* fftools/ffmpeg_filter: stop taking display matrix from global side dataAnton Khirnov2024-02-211-7/+0
| | | | | It should never be necessary now that decoders propagate global side data to frames.
* avutil/version: Remove outdated checksAndreas Rheinhardt2024-02-211-16/+0
| | | | Signed-off-by: Andreas Rheinhardt <[email protected]>
* avcodec: Remove superfluous ';' outside of functionsAndreas Rheinhardt2024-02-214-53/+53
| | | | | | | | | Inside a function an extra ';' is a null statement; but outside of it it simply must not happen. So remove them. Reviewed-by: Nuo Mi <[email protected]> Signed-off-by: Andreas Rheinhardt <[email protected]>
* avcodec/vvc/vvcdsp: Remove pointless wrappersAndreas Rheinhardt2024-02-212-19/+1
| | | | | Reviewed-by: Nuo Mi <[email protected]> Signed-off-by: Andreas Rheinhardt <[email protected]>
* avcodec/vvc/vvc_ps: Use union for luts to avoid unaligned accessesAndreas Rheinhardt2024-02-216-13/+14
| | | | | | | | | | These arrays are currently accessed via uint16_t* pointers although nothing guarantees their alignment. Furthermore, this is problematic wrt the effective-type rules. Fix this by using a union of arrays of uint8_t and uint16_t. Reviewed-by: Nuo Mi <[email protected]> Signed-off-by: Andreas Rheinhardt <[email protected]>
* avcodec/vvc/vvc_ps: Check before accessAndreas Rheinhardt2024-02-211-1/+1
| | | | | | | | | | | | | max_bin_idx can be at most LMCS_MAX_BIN_SIZE - 1 here, so pivot[LCMS_MAX_BIN_SIZE + 1] may be accessed, but pivot has only LCMS_MAX_BIN_SIZE + 1 elements (unless the values of pivot were so that it is always assured that pivot[LCMS_MAX_BIN_SIZE] is always < sample (which it is iff it is always < 2^bit_depth - 1)). So reorder the checks. Reviewed-by: Nuo Mi <[email protected]> Signed-off-by: Andreas Rheinhardt <[email protected]>