[FFmpeg-devel] [PATCH] ffmpeg_hw: make hardware selection for filters more user friendly

2021-06-28 Thread Haihao Xiang
When a device is derived from a source device, there are at least 2 devices, and usually the derived device is the expected device, so let's pick the last device if user doesn't specify the filter device with filter_hw_device option After applying this patch, the command below can work: $> ffmpeg

Re: [FFmpeg-devel] [PATCH] avfilter/vf_libvmaf: update filter for libvmaf v2.0.0

2021-06-28 Thread Kyle Swanson
Hi, Updated patch attached. Thank you to Christopher Degawa for testing and providing feedback. Kyle 0001-avfilter-vf_libvmaf-update-filter-for-libvmaf-v2.0.0.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https:/

[FFmpeg-devel] Compensating for mistakes of video encoder rtsp server implementations

2021-06-28 Thread Hayden Myers
How does one approach the problem of an encoder's incorrect implementation of the rtsp protocol? My non-working example here is a Coretec mpeg-4 encoder. I'm not sure the manufacturer is still in business, but they're still in use. Using rtsp, the encoder sends no response to a setup message.

Re: [FFmpeg-devel] [PATCH] mxfdec.c: prefer metadata from Footer

2021-06-28 Thread emcodem
Am 2021-06-28 03:00, schrieb Marton Balint: On Sun, 27 Jun 2021, emco...@ffastrans.com wrote: Am 2021-06-27 20:12, schrieb Marton Balint: Why? I though it is enough if you store the partition number in the metadata set, that way you should be able to compare if the existing metadata set is bet

Re: [FFmpeg-devel] av_packet_get_side_data() declaration in libavcodec/packet.h

2021-06-28 Thread Hendrik Leppkes
On Mon, Jun 28, 2021 at 5:45 PM Gerd Röthig wrote: > > Hello all, > > Trying to compile the Chromium browser by myself, I encountered that compile > fails due to variable > declarations in ffmpeg_demuxer.cc in Chromium's source code. > > In the file, the following call to av_packet_get_side_data(

Re: [FFmpeg-devel] [PATCH] avcodec/mjpegdec: Adjustable discard threshold

2021-06-28 Thread Michael Niedermayer
On Sun, Jun 27, 2021 at 01:38:01PM -0400, Andriy Gelman wrote: > On Sun, 27. Jun 15:53, Michael Niedermayer wrote: > > Fixes regression > > Fixes: last frame of Ticket9287 > > > > Analysed-by: Andriy Gelman > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/mjpegdec.c | 2 +- > > 1 fi

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/clearvideo: Check tile_size to be not too large

2021-06-28 Thread Michael Niedermayer
On Mon, Jun 21, 2021 at 11:25:42PM +0200, Michael Niedermayer wrote: > Fixes: left shift of 1 by 31 places cannot be represented in type 'int' > Fixes: > 35023/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_CLEARVIDEO_fuzzer-6740166587842560 > > Found-by: continuous fuzzing process > https://

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/utils: Use 64bit for intermediate in AV_CODEC_ID_ADPCM_THP* duration calculation

2021-06-28 Thread Michael Niedermayer
On Thu, Jun 24, 2021 at 10:57:24PM +0200, Michael Niedermayer wrote: > Fixes: signed integer overflow: 486539264 * 14 cannot be represented in type > 'int' > Fixes: > 35281/clusterfuzz-testcase-minimized-ffmpeg_dem_RSD_fuzzer-6068262742917120 > > Found-by: continuous fuzzing process > https://g

Re: [FFmpeg-devel] [PATCH 2/3] avformat/sbgdec: Check opt_duration and start for overflow

2021-06-28 Thread Michael Niedermayer
On Sun, Jun 27, 2021 at 08:22:12PM +0200, Nicolas George wrote: > James Almer (12021-06-25): > > Afaics, it checks the very first character to be < '0' || > '9' for both > > hours and minutes, so strtol() is not going to see a '-'. > > Is there supposed to be one for seconds in valid files? If not,

[FFmpeg-devel] av_packet_get_side_data() declaration in libavcodec/packet.h

2021-06-28 Thread Gerd Röthig
Hello all, Trying to compile the Chromium browser by myself, I encountered that compile fails due to variable declarations in ffmpeg_demuxer.cc in Chromium's source code. In the file, the following call to av_packet_get_side_data() is made: size_t id_size = 0; uint8_t* id_data = av_pac

Re: [FFmpeg-devel] [PATCH] libswscale: Make sws_init_context thread safe.

2021-06-28 Thread Peter Lundblad
Hi, Would anyone consider applying this fairly simple patch to fix a multi-threading issue in the sws initialization code? Regards, //Peter On Wed, Apr 28, 2021 at 11:51 AM Peter Lundblad wrote: > Call ff_sws_rgb2rgb_init via ff_thread_once instead of checking one of the > variables it updates

Re: [FFmpeg-devel] [PATCH] avcodec/h264_metadata_bsf: Allow zeroing constraint_set4_flag and constraint_set5_flag

2021-06-28 Thread Derek Buitenhuis
On 6/27/2021 12:56 PM, Derek Buitenhuis wrote: > On 6/24/2021 2:21 PM, Derek Buitenhuis wrote: >> I'll wait a bit and see if he has any. > > Ping. Will push tonight or tomorrow with typo fixes + lavc version bump if there are no objections. - Derek ___

Re: [FFmpeg-devel] [PATCH] lavd/lavfi.c: Set time_base for 608 cc to container time_base.

2021-06-28 Thread Thilo Borgmann
Hi, > when transcoding 608 cc, the cc stream frame pts is set to the same value as > its container frame's pts. However, the time_base is always set to 1/9 > (default) in the initialization stage. Which causes timing issues when the > container time_base is actually not 1/9. identical

Re: [FFmpeg-devel] [PATCH v2] avformat: add a concat protocol that takes a line break delimited list of resources

2021-06-28 Thread Nicolas George
James Almer (12021-06-27): > Your first paragraph started with "Which characters must be escaped?" Yes, but it was a follow-up to "the doc is not detailed enough". I trim my mails, because it makes reading easier, but the context still matters. Should I leave more of it, even for conversations ove

Re: [FFmpeg-devel] Unable to use sqrt() and exp() in CUDA

2021-06-28 Thread Dylan Fernando
I've only been including vector_helpers.cuh in the .cu file. I think cuda.h is being included because I include libavutil/hwcontext_cuda_internal.h in the c file. I was looking at vf_thumbnail_cuda.c for reference and I copied the line #include "libavutil/hwcontext_cuda_internal.h". __