18:48
avfilter/avfilter: make filter_activate_default request frames on behalf of sinks Sinks without an activate callback have no means to request frames in their input, therefore the default activate callback should do it for them. Fixes ticket #11624. Fixes ticket #10988. Fixes ticket #10990. Signed-off-by: Marton Balint <cusⓐpasswd.hu> [DH] libavfilter/avfilter.c
14:28
avfilter/buffersink: keep requesting frames if one activation of the graph does not provide one A frame graph activation might not produce a frame in the requested sink, so keep on requesting a frame there unless we encounter a filter activation with buffersrc empty error. This makes av_buffersink_get_frame(_flags) work according to its documentation which claims that EAGAIN is only returned if additional frames must be inserted into the graph. Fate changes are because audio frames will have different sizes at segment boundaries, but content is the same. Signed-off-by: Marton Balint (...)
13:50
avfilter: signal an empty buffersrc with an explicit activate error code No change in functionality. Signed-off-by: Marton Balint <cusⓐpasswd.hu> [DH] libavfilter/avfiltergraph.c [DH] libavfilter/buffersink.c [DH] libavfilter/buffersrc.c [DH] libavfilter/filters.h
13:39
avfilter/avfilter: always forward request frame in filter_activate_default Even if all inputs are blocked an activate callback should request a frame on some if its inputs if a frame is requested on any of its outputs. Signed-off-by: Marton Balint <cusⓐpasswd.hu> [DH] libavfilter/avfilter.c
06:19
fate/filter-video: add ffprobe test for dual output select filter Signed-off-by: Marton Balint <cusⓐpasswd.hu> [DH] tests/fate-run.sh [DH] tests/fate/filter-video.mak [DH] tests/ref/fate/filter-select-ffprobe
21:25
avcodec/ffv1enc: Use dummies to avoid UB pointer arithmetic Fixes the following FATE-tests when run under Clang-UBSan: ffmpeg-loopback-decoding, lavf-mxf_ffv1, vsynth1,2,3,_lena-ffv1-v0,2, vsynth1-ffv1,2,3,_lena, vsynth1,2,3,_lena-ffv1-v3-yuv420p,422p10,444p16 Signed-off-by: Andreas Rheinhardt <andreas.rheinhardtⓐoutlook.com> [DH] libavcodec/ffv1enc.c [DH] libavcodec/ffv1enc_template.c
20:45
avcodec/indeo3: Constify ref_block in decode_cell_data() Also use smaller scope while just at it. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardtⓐoutlook.com> [DH] libavcodec/indeo3.c
20:39
avcodec/indeo3: Consistently use ptrdiff_t for strides Signed-off-by: Andreas Rheinhardt <andreas.rheinhardtⓐoutlook.com> [DH] libavcodec/indeo3.c
20:36
avcodec/indeo3: Fix UB pointer arithmetic Fixes the following error when running with Clang-UBSan: src/libavcodec/indeo3.c:556:26: runtime error: applying non-zero offset 2560 to null pointer This fixes the indeo3-2 FATE test. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardtⓐoutlook.com> [DH] libavcodec/indeo3.c
07:10
avcodec/dfpwmenc: Remove write-only context member Signed-off-by: Andreas Rheinhardt <andreas.rheinhardtⓐoutlook.com> [DH] libavcodec/dfpwmenc.c
05:37
avcodec/dfpwmenc: Correctly pad input Before this patch, the DFPWM1a encoder was marked as supporting variable frame sizes. The DFPWM1a format converts eight bytes of input into one output byte and so it simply padded the number of data output by frame->nb_samples * frame->ch_layout.nb_channels / 8 + (frame->nb_samples % 8 > 0 ? 1 : 0) This has several bugs: a) The additional byte leads to eight additional input byte being read; this can read into the frame's padding, i.e. the data can be uninitialized. b) The criterion for whether one should pad is wrong: nb_samples * (...)
13:51
avcodec/mss2dsp: use FF_PTR_ADD to add offsets to a pointer Fixes: libavcodec/mss2dsp.c:59:14: runtime error: applying zero offset to null pointer Tested-by: Kacper Michajlow <kasper93ⓐgmail.com> Signed-off-by: James Almer <jamrialⓐgmail.com> [DH] libavcodec/mss2dsp.c
02:22
avfilter/x86/f_ebur128: replace AVX2 instruction with AVX equivalent Using vpbroadcastq in an AVX function will result in SIGILL errors on pre Haswell/Zen processors. Signed-off-by: James Almer <jamrialⓐgmail.com> [DH] libavfilter/x86/f_ebur128.asm