Re: [FFmpeg-devel] Patch for seg fault in swr_convert_internal() -> sum2_float during dithering

2018-04-07 Thread Hendrik Schreiber
> On Apr 7, 2018, at 02:07, Michael Niedermayer wrote: > > On Thu, Apr 05, 2018 at 02:38:03PM +0200, Hendrik Schreiber wrote: > > will apply > > thanks Great. Much appreciated. -hendrik ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://f

Re: [FFmpeg-devel] lavfi/silencedetect v3

2018-04-07 Thread Paul B Mahol
On 4/7/18, James Almer wrote: > On 4/1/2018 8:26 AM, Paul B Mahol wrote: >> On 3/5/18, Gaullier Nicolas wrote: >>> Hello, >>> I have not received any comment yet on my patchset v3 ("add mono mode" >>> new >>> feature + 4 fixes including a ticket), does it look good to you, could it >>> be >>> com

[FFmpeg-devel] [PATCH] gitattributes - force LF for some FATE srt and webvtt refs

2018-04-07 Thread Gyan Doshi
Please first read the commit msg below, which is, I hope, self-explanatory, but I want to note a few things. When searching to see how this issue was addressed in the past, I saw the recommendation was to set git global autocrlf to false. That's inadvisable because it affects checkout/clones o

Re: [FFmpeg-devel] [PATCH] avformat/rawenc: check stream type

2018-04-07 Thread Carl Eugen Hoyos
2018-04-04 12:23 GMT+02:00, Gyan Doshi : > Validate codec of stream to be muxed except for data muxer. What issue does your patch fix? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avformat/rawenc: check stream type

2018-04-07 Thread Gyan Doshi
On 4/7/2018 4:35 PM, Carl Eugen Hoyos wrote: 2018-04-04 12:23 GMT+02:00, Gyan Doshi : Validate codec of stream to be muxed except for data muxer. What issue does your patch fix? Had an user on SE sometime back extract a video stream to a .h264 (in order to retime PTS). The retiming didn't w

[FFmpeg-devel] Add IRC nicknames to MAINTAINERS?

2018-04-07 Thread Tomas Härdin
Hi It's come up a couple of times that IRC nicknames would be useful. I at least tend to answer IRC much more quickly than email. Something like patch attached maybe? /TomasFrom 03225dda47c73c3323c3276353d0ac896123cd3f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tomas=20H=C3=A4rdin?= Date: Sat, 7 A

Re: [FFmpeg-devel] Add IRC nicknames to MAINTAINERS?

2018-04-07 Thread James Almer
On 4/7/2018 9:08 AM, Tomas Härdin wrote: > Hi > > It's come up a couple of times that IRC nicknames would be useful. I at > least tend to answer IRC much more quickly than email. Something like > patch attached maybe? > > /Tomas > > > 0001-Add-IRC-nicknames.patch > > > From 03225dda47c73c3323

Re: [FFmpeg-devel] [PATCH] libavcodec/rv34: error out earlier on missing references

2018-04-07 Thread Michael Niedermayer
On Mon, Apr 02, 2018 at 11:24:59PM +0200, Michael Niedermayer wrote: > Fixes visual corruption on seeking > > Fixes: downloadTest_clip_24M.rmvb > > Signed-off-by: Michael Niedermayer > --- > libavcodec/rv34.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) will apply [...] -- M

Re: [FFmpeg-devel] [PATCH] avformat/rawenc: check stream type

2018-04-07 Thread Carl Eugen Hoyos
2018-04-07 13:32 GMT+02:00, Gyan Doshi : > On 4/7/2018 4:35 PM, Carl Eugen Hoyos wrote: >> 2018-04-04 12:23 GMT+02:00, Gyan Doshi : >>> Validate codec of stream to be muxed except for data muxer. >> >> What issue does your patch fix? > > Had an user on SE sometime back extract a video stream to a .

[FFmpeg-devel] [PATCH]lavc/clearvideo: Allow decoding without extradata

2018-04-07 Thread Carl Eugen Hoyos
Hi! Attached patch fixes (initial) playback for the files in http://samples.ffmpeg.org/V-codecs/UCOD/noextradata/ - kindly provided by Piotr. Please comment, Carl Eugen From 6fb8df8d4750e66d62eb75ed57ea56463c4ebd38 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Sat, 7 Apr 2018 21:30:45 +0

Re: [FFmpeg-devel] [PATCH]lavc/clearvideo: Allow decoding without extradata

2018-04-07 Thread Paul B Mahol
On 4/7/18, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes (initial) playback for the files in > http://samples.ffmpeg.org/V-codecs/UCOD/noextradata/ > - kindly provided by Piotr. > > Please comment, Carl Eugen > LGTM Files are probably invalid, but whatever. _

Re: [FFmpeg-devel] [PATCH]lavc/clearvideo: Allow decoding without extradata

2018-04-07 Thread Carl Eugen Hoyos
2018-04-07 22:03 GMT+02:00, Paul B Mahol : > On 4/7/18, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch fixes (initial) playback for the files in >> http://samples.ffmpeg.org/V-codecs/UCOD/noextradata/ >> - kindly provided by Piotr. >> >> Please comment, Carl Eugen >> > > LGTM Patch applied,

[FFmpeg-devel] [PATCH] configure: disable direct stripping in OpenBSD

2018-04-07 Thread James Almer
It appears strip -o creates new files without preserving permissions from the source binary, resulting in non executable files. Signed-off-by: James Almer --- Untested. This is purely based on what i see in http://fate.ffmpeg.org/ Alternatively, although probably much harder to do, would be to m

[FFmpeg-devel] [PATCH 2/2] lavf/format: Remove the dead code in av_probe_input_buffer2.

2018-04-07 Thread Jun Zhao
From df6293f42fad1e4521389c23c984519caa98d3f5 Mon Sep 17 00:00:00 2001 From: Jun Zhao Date: Sun, 8 Apr 2018 08:09:13 +0800 Subject: [PATCH 2/2] lavf/format: Remove the dead code in av_probe_input_buffer2. Remove the dead code in av_probe_input_buffer2 Signed-off-by: Jun Zhao --- libavformat/

[FFmpeg-devel] [PATCH 1/2] avformat/avio: make the logic simple

2018-04-07 Thread Jun Zhao
From 733dfb98152a7a629e0440f6f807f76c2b2a9584 Mon Sep 17 00:00:00 2001 From: Jun Zhao Date: Sun, 8 Apr 2018 08:05:08 +0800 Subject: [PATCH 1/2] avformat/avio: make the logic simple remove the "ret" to make the code simple and generic. Signed-off-by: Jun Zhao --- libavformat/avio.c | 5 ++---

[FFmpeg-devel] [PATCH] avcodec/movtextdec: Check style_start/end

2018-04-07 Thread Michael Niedermayer
Limits based on 3GPP TS 26.245 V14.0.0 Fixes: Timeout Fixes: 6377/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_MOVTEXT_fuzzer-5175929115508736 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavco

Re: [FFmpeg-devel] [PATCH] avcodec/movtextdec: Check style_start/end

2018-04-07 Thread Philip Langdale
On Sun, 8 Apr 2018 03:29:44 +0200 Michael Niedermayer wrote: > Limits based on 3GPP TS 26.245 V14.0.0 > Fixes: Timeout > Fixes: > 6377/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_MOVTEXT_fuzzer-5175929115508736 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/

Re: [FFmpeg-devel] [PATCH v2] lavc/hevc: Don't parse NAL unit for a dummy buffer

2018-04-07 Thread Xiang, Haihao
Hi Steven, Are there more comments on this patch? Thanks Haihao > hevc parser mistakenly reports the following message if a dummy buffer > is padded for EOF > >[hevc @ 0x559b63848610] missing picture in access unit > > v2: use the preferred code style and rebase the patch > > Signed-o

Re: [FFmpeg-devel] [PATCH] avformat/rawenc: check stream type

2018-04-07 Thread Gyan Doshi
On 4/8/2018 1:19 AM, Carl Eugen Hoyos wrote: 2018-04-07 13:32 GMT+02:00, Gyan Doshi : Had an user on SE sometime back extract a video stream to a .h264 (in order to retime PTS). Does this work at all? With input -r, yes. For streams with b-pyramid, mp4box does. A warning would be suffi

Re: [FFmpeg-devel] [PATCH v2] lavc/hevc: Don't parse NAL unit for a dummy buffer

2018-04-07 Thread Steven Liu
> On 8 Apr 2018, at 12:53, Xiang, Haihao wrote: > > > > Hi Steven, > > Are there more comments on this patch? > > Thanks > Haihao > > >> hevc parser mistakenly reports the following message if a dummy buffer >> is padded for EOF >> >> [hevc @ 0x559b63848610] missing picture in access

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: fix handling of delete_segments when %v is present

2018-04-07 Thread Steven Liu
> On 6 Apr 2018, at 18:27, Bodecs Bela wrote: > > Dear All, > > when var_stream_map option is used, %v must appear either in segment name > template or in the directory path. This latter case currently is not handled > and using delete_segments flag of hls_flags is broken now. This patch fix