[FFmpeg-devel] [PATCH V1 1/2] lavf/avio: remove ffio_open2_wrapper function

2019-07-12 Thread Jun Zhao
From: Jun Zhao Remove the function ffio_open2_wrapper, it's not being used anymore. Signed-off-by: Jun Zhao --- libavformat/aviobuf.c |6 -- libavformat/internal.h |3 --- 2 files changed, 0 insertions(+), 9 deletions(-) diff --git a/libavformat/aviobuf.c b/libavformat/aviobuf.c

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg_filter: use -reinit_filter to disable/enable auto scale

2019-07-12 Thread Eoff, Ullysses A
> > > Add an option to disable/enable "auto insert" is Ok for me, but I > > > think if you reuse the -reinit_filter option, you need to update doc > > > part at the same time. > > > > I prefer to add a separate option, too. > > Depending on the comments, I'll either update doc for reinit_filter,

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-12 Thread Eoff, Ullysses A
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > Michael Niedermayer > Sent: Friday, July 12, 2019 11:21 AM > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter > > > On Wed,

Re: [FFmpeg-devel] [PATCH v3 3/3] lavc/libdavs2.c: use decoder data directly instead of memcpy

2019-07-12 Thread hwren
At 2019-07-13 01:38:55, "Marton Balint" wrote: > > >On Fri, 12 Jul 2019, hwrenx wrote: > >> Can effectivly improved decoding speed when memcpy becomes a limitation >> for proccessing high resolution source. >> Tested under i7-8700k with `ffmpeg -i 7680x4320.avs2 -vsync 0 -f null -` >> got

[FFmpeg-devel] [PATCH] dump_extra: Don't add extradata if it already exists

2019-07-12 Thread Andreas Rheinhardt
The dump_extra bitstream filter currently simply adds the extradata to the packets indicated by the user without checking whether said extradata already exists in the packets. Besides wasting space duplicated extradata in the same packet/access unit is also forbidden for some codecs, e.g. MPEG-2.

Re: [FFmpeg-devel] [PATCH] Add support for Display Definition Segment to DVB Subtitle

2019-07-12 Thread Moritz Barsnick
On Sat, Jul 13, 2019 at 00:24:29 +0200, mikroh...@gmail.com wrote: > From: Jernej I think you had your full name in here originally. Note that whatever you provide here will be in the repo forever. > > Patch with requested changes. Your original commit text should be in here, not your message

Re: [FFmpeg-devel] [PATCH v2] fftools/ffmpeg: fix video frame lost sync for the dual input ts files with audio

2019-07-12 Thread Limin Wang
On Fri, Jul 12, 2019 at 09:46:01PM +0200, Michael Niedermayer wrote: > On Thu, Jul 11, 2019 at 09:54:29AM +0800, lance.lmw...@gmail.com wrote: > > From: Limin Wang > > > > How to reproduce the problem(use two ts files with audio stream): > > ffmpeg -i left_w_a.ts -i right_w_a.ts -filter_complex

Re: [FFmpeg-devel] [PATCH v1 2/2] libavformat/file: initilize the fd to -1 instead of 0(valid fd) in case unexpected file close

2019-07-12 Thread Limin Wang
On Fri, Jul 12, 2019 at 10:44:48PM +0200, Michael Niedermayer wrote: > On Tue, Jun 18, 2019 at 06:45:13PM +0800, lance.lmw...@gmail.com wrote: > > From: Limin Wang > > > > Signed-off-by: Limin Wang > > --- > > libavformat/file.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git

[FFmpeg-devel] [PATCH] Add support for Display Definition Segment to DVB Subtitle

2019-07-12 Thread mikrohard
From: Jernej Patch with requested changes. --- libavcodec/dvbsub.c | 19 --- 1 file changed, 16 insertions(+), 3 deletions(-) diff --git a/libavcodec/dvbsub.c b/libavcodec/dvbsub.c index 8cce702..a8d43d8 100644 --- a/libavcodec/dvbsub.c +++ b/libavcodec/dvbsub.c @@ -247,9

Re: [FFmpeg-devel] [PATCH 2/2] mpeg4_unpack_bframes: Merge close and flush

2019-07-12 Thread Michael Niedermayer
On Wed, Jul 10, 2019 at 11:08:27PM +0200, Andreas Rheinhardt wrote: > They are identical now. > > Signed-off-by: Andreas Rheinhardt > --- > libavcodec/mpeg4_unpack_bframes_bsf.c | 12 +++- > 1 file changed, 3 insertions(+), 9 deletions(-) > > diff --git

Re: [FFmpeg-devel] [PATCH] libavfilter: add photosensitivity filter

2019-07-12 Thread Michael Niedermayer
On Fri, Jul 12, 2019 at 11:26:34AM +0200, Paul B Mahol wrote: > From: Vladimir Panteleev > > Signed-off-by: Paul B Mahol > --- > doc/filters.texi | 20 ++ > libavfilter/Makefile | 1 + > libavfilter/allfilters.c | 1 + >

Re: [FFmpeg-devel] [PATCH v1 1/2] libavformat/avio: Fix for the unexpected file close which will cause fd 0 is closed

2019-07-12 Thread Michael Niedermayer
On Tue, Jun 18, 2019 at 06:45:12PM +0800, lance.lmw...@gmail.com wrote: > From: Limin Wang > > Signed-off-by: Limin Wang > --- > libavformat/avio.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/libavformat/avio.c b/libavformat/avio.c > index 663789ec02..c1d10ee8c3 100644 > ---

Re: [FFmpeg-devel] [PATCH v1 2/2] libavformat/file: initilize the fd to -1 instead of 0(valid fd) in case unexpected file close

2019-07-12 Thread Michael Niedermayer
On Tue, Jun 18, 2019 at 06:45:13PM +0800, lance.lmw...@gmail.com wrote: > From: Limin Wang > > Signed-off-by: Limin Wang > --- > libavformat/file.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavformat/file.c b/libavformat/file.c > index 08c7f8e6dd..40ae9ad2a8 100644 > ---

Re: [FFmpeg-devel] [PATCH v2] fftools/ffmpeg: fix video frame lost sync for the dual input ts files with audio

2019-07-12 Thread Michael Niedermayer
On Thu, Jul 11, 2019 at 09:54:29AM +0800, lance.lmw...@gmail.com wrote: > From: Limin Wang > > How to reproduce the problem(use two ts files with audio stream): > ffmpeg -i left_w_a.ts -i right_w_a.ts -filter_complex "hstack=inputs=2" -f > null - > > With above command, the audio stream of the

Re: [FFmpeg-devel] [PATCHv2] avformat/movenc: use unspecified language by default

2019-07-12 Thread Marton Balint
On Thu, 11 Jul 2019, Marton Balint wrote: On Tue, 9 Jul 2019, Marton Balint wrote: English was used before. Will apply this soon. Applied. Regards, Marton ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] avformat/mpegts: respect program number when merging streams

2019-07-12 Thread Marton Balint
On Tue, 9 Jul 2019, Aman Gupta wrote: On Tue, Jul 9, 2019 at 2:07 PM Marton Balint wrote: merge_pmt_versions was not usable if multiple programs were present because when it was searching for candidate streams it did not make sure that the PMT was of the same program. This caused the

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/pngdec: Check that previous_picture has same w/h/format

2019-07-12 Thread Michael Niedermayer
On Fri, Jul 12, 2019 at 10:30:26AM +0200, Paul B Mahol wrote: > On 7/11/19, Michael Niedermayer wrote: > > Fixes: out of array access > > Fixes: > > 15540/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_APNG_fuzzer-5684905029140480 > > > > Found-by: continuous fuzzing process > >

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/huffyuvdec: Check vertical subsampling in hymt

2019-07-12 Thread Michael Niedermayer
On Fri, Jul 12, 2019 at 10:35:41AM +0200, Paul B Mahol wrote: > LGTM will apply thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The greatest way to live with honor in this world is to be what we pretend to be. -- Socrates signature.asc Description:

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/huffyuv: remove gray8a (the format is listed but not supported by the implementation)

2019-07-12 Thread Michael Niedermayer
On Fri, Jul 12, 2019 at 10:41:19AM +0200, Paul B Mahol wrote: > LGTM will apply thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB It is what and why we do it that matters, not just one of them. signature.asc Description: PGP signature

Re: [FFmpeg-devel] [PATCH] libavfilter: add photosensitivity filter

2019-07-12 Thread Marton Balint
On Fri, 12 Jul 2019, Paul B Mahol wrote: From: Vladimir Panteleev I guess you should mention ticket #2104. Regards, Marton Signed-off-by: Paul B Mahol --- doc/filters.texi | 20 ++ libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 +

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-12 Thread Michael Niedermayer
On Wed, Jul 10, 2019 at 07:54:32PM +, Eoff, Ullysses A wrote: > > -Original Message- > > From: Eoff, Ullysses A > > Sent: Tuesday, July 09, 2019 6:45 PM > > To: ffmpeg-devel@ffmpeg.org > > Cc: Eoff, Ullysses A > > Subject: [PATCH] avfilter: add rawdump filter > > > > This filter

Re: [FFmpeg-devel] [PATCH] Add support for Display Definition Segment to DVB Subtitle encoder

2019-07-12 Thread Marton Balint
On Fri, 12 Jul 2019, Jernej wrote: Here's the fixed patch. I'm not familiar with providing patches via a mailing list. If replying doesn't work, I'll just use a new git send-mail. Yes, please, preferably with --in-reply-to= Thanks, Marton Signed-off-by: Jernej Fijacko ---

Re: [FFmpeg-devel] [PATCH 4/4] avcodec/dirac_parser: Fix overflow in dts

2019-07-12 Thread Michael Niedermayer
On Thu, Jul 11, 2019 at 08:58:50PM -0300, James Almer wrote: > On 7/11/2019 6:49 PM, Michael Niedermayer wrote: > > Fixes: signed integer overflow: -2147483648 - 1 cannot be represented in > > type 'int' > > Fixes: > >

Re: [FFmpeg-devel] [PATCH v3 3/3] lavc/libdavs2.c: use decoder data directly instead of memcpy

2019-07-12 Thread Marton Balint
On Fri, 12 Jul 2019, hwrenx wrote: Can effectivly improved decoding speed when memcpy becomes a limitation for proccessing high resolution source. Tested under i7-8700k with `ffmpeg -i 7680x4320.avs2 -vsync 0 -f null -` got performance 23fps => 42fps Signed-off-by: hwrenx ---

Re: [FFmpeg-devel] [PATCH] lavc: Add libsvt-av1 encoder wrapper

2019-07-12 Thread Li, Zhong
> You (or anyone) are welcome to contribute to the SVT-AV1 project, and > submit the FFmpeg plugin changes you'd like to make here: > https://github.com/OpenVisualCloud/SVT-AV1/tree/master/ffmpeg_plugin, > and we will take your commits seriously. But I have not got any code > changes made by you

[FFmpeg-devel] [PATCH v3 3/3] lavc/libdavs2.c: use decoder data directly instead of memcpy

2019-07-12 Thread hwrenx
Can effectivly improved decoding speed when memcpy becomes a limitation for proccessing high resolution source. Tested under i7-8700k with `ffmpeg -i 7680x4320.avs2 -vsync 0 -f null -` got performance 23fps => 42fps Signed-off-by: hwrenx --- libavcodec/libdavs2.c | 54

[FFmpeg-devel] [PATCH v3 1/3] lavc/avs2_parser.c: fix for missing start code

2019-07-12 Thread hwrenx
Signed-off-by: hwrenx --- libavcodec/avs2_parser.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavcodec/avs2_parser.c b/libavcodec/avs2_parser.c index 1c9b342..dc2fa16 100644 --- a/libavcodec/avs2_parser.c +++ b/libavcodec/avs2_parser.c @@ -23,8 +23,10 @@ #define

[FFmpeg-devel] [PATCH v3 2/3] lavc/libdavs2.c: change decoder info level

2019-07-12 Thread hwrenx
Signed-off-by: hwrenx --- libavcodec/libdavs2.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavcodec/libdavs2.c b/libavcodec/libdavs2.c index 218f3ec..1b274a3 100644 --- a/libavcodec/libdavs2.c +++ b/libavcodec/libdavs2.c @@ -44,7 +44,9 @@ static av_cold int

[FFmpeg-devel] [PATCH v3 0/3] optimize frame dumping and fix missing start codes

2019-07-12 Thread hwrenx
The previous version 2 patches were deprecated for using pointer to itself can be unsafe when release frames. This patch set is mainly about avoiding using memcpy when fetching frame data, and fixed missing start codes which may cause error packetization sometime. Default info level of libdavs2

Re: [FFmpeg-devel] [PATCH] lavc: Add libsvt-av1 encoder wrapper

2019-07-12 Thread Sun, Jing A
-Original Message- From: Li, Zhong Sent: Friday, July 12, 2019 6:32 PM To: ffmpeg-devel@ffmpeg.org Cc: Daryl Seah ; Li, Zhong ; Daryl; Seah; Sun, Jing A ; Tang, Zhizhen Subject: [PATCH] lavc: Add libsvt-av1 encoder wrapper From: Daryl Seah Signed-off-by: Daryl Seah Signed-off-by:

[FFmpeg-devel] [PATCH] Display statistics on decode error

2019-07-12 Thread Rémi Achard
--- fftools/ffmpeg.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index 01f04103cf..43bd47ded4 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -2108,8 +2108,10 @@ static void check_decode_result(InputStream *ist, int

[FFmpeg-devel] [PATCH v2 3/3] lavc/libdavs2.c: use decoder data directly instead of memcpy

2019-07-12 Thread hwrenx
Signed-off-by: hwrenx --- libavcodec/libdavs2.c | 55 ++- 1 file changed, 32 insertions(+), 23 deletions(-) diff --git a/libavcodec/libdavs2.c b/libavcodec/libdavs2.c index df538af..f88dfa0 100644 --- a/libavcodec/libdavs2.c +++

[FFmpeg-devel] [PATCH v2 2/3] lavc/libdavs2.c: change info level, only print errors

2019-07-12 Thread hwrenx
Signed-off-by: hwrenx --- libavcodec/libdavs2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/libdavs2.c b/libavcodec/libdavs2.c index 218f3ec..df538af 100644 --- a/libavcodec/libdavs2.c +++ b/libavcodec/libdavs2.c @@ -44,7 +44,7 @@ static av_cold int

[FFmpeg-devel] [PATCH v2 1/3] lavc/avs2_parser.c: fix for missing start code

2019-07-12 Thread hwrenx
Signed-off-by: hwrenx --- libavcodec/avs2_parser.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavcodec/avs2_parser.c b/libavcodec/avs2_parser.c index 1c9b342..dc2fa16 100644 --- a/libavcodec/avs2_parser.c +++ b/libavcodec/avs2_parser.c @@ -23,8 +23,10 @@ #define

Re: [FFmpeg-devel] [PATCH] Display decoded frame number on error

2019-07-12 Thread Remi Achard
> > This did the trick thanks. For the -v verbose this indeed print the last > decoded frame on normal run but not when using -xerror. > Eg. > [AVIOContext @ 0x7f7f7780] Statistics: 52700397 bytes read, 4 seeks > I guess because print_report is not called when exit_program is called in

Re: [FFmpeg-devel] [PATCH 2/3] lavf/webm_chunk: Fix NULL dereference

2019-07-12 Thread Andreas Rheinhardt
Paul B Mahol: > On 7/12/19, Andreas Rheinhardt wrote: >> Andreas Rheinhardt: >>> Andreas Rheinhardt: Andreas Rheinhardt: > Andreas Rheinhardt: >> The earlier version of the webm_chunk muxer had several bugs: >> >> 1. If the first packet of an audio stream didn't have a PTS of

Re: [FFmpeg-devel] [PATCH] Display decoded frame number on error

2019-07-12 Thread Remi Achard
> > Add `-copyts` for that. If you want the index of the first decode > failure, you don't need this patch. Run with `-xerror -v verbose`. > ist->frames_decoded will be printed for each input stream. Add 1 to > that. The `time=` value at the end should be the timestamp + duration of > the last

Re: [FFmpeg-devel] [PATCH] Display decoded frame number on error

2019-07-12 Thread Gyan
On 12-07-2019 04:08 PM, Remi Achard wrote: Which is good: that means you get the same number with or without it. Sorry I was unclear, I mean that best_effort_timestamp is relative to trim start. Would be nice to have the abosulte timestamp in the file, regardless of where the -ss flag starts

Re: [FFmpeg-devel] [PATCH] Display decoded frame number on error

2019-07-12 Thread Remi Achard
> Which is good: that means you get the same number with or without it. > Sorry I was unclear, I mean that best_effort_timestamp is relative to trim start. Would be nice to have the abosulte timestamp in the file, regardless of where the -ss flag starts no ?

Re: [FFmpeg-devel] [PATCH] Display decoded frame number on error

2019-07-12 Thread Nicolas George
Remi Achard (12019-07-12): > Would you advice using the AVFrame::best_effort_timestamp field ? That is the one. > One problem I see is that it doesn't seems to be aware of trim command (eg. > -ss flag). Which is good: that means you get the same number with or without it. Regards, --

[FFmpeg-devel] [PATCH] lavc: Add libsvt-av1 encoder wrapper

2019-07-12 Thread Zhong Li
From: Daryl Seah Signed-off-by: Daryl Seah Signed-off-by: Jing SUN Signed-off-by: ZhiZhen Tang Signed-off-by: Zhong Li --- It is part of requirement from ticket #7914. Since SVT-AV1 library is not stable, this patch is not ready to be merged but just call for comments of this wrapper and SVT

Re: [FFmpeg-devel] [PATCH] Display decoded frame number on error

2019-07-12 Thread Remi Achard
> > Exactly, you need to use the format string for uint64_t, PRIu64 from > memory; check in the code base and standard. > Thanks for the pointer, didn't knew this. > Then I think the timestamp of the frame would be a better choice. IIUC, > the field you print depends on the way ffmpeg processes

Re: [FFmpeg-devel] [PATCH] Display decoded frame number on error

2019-07-12 Thread Remi Achard
> > Let's call good frames G and bad frames B. If your input isG G G B B > B G G B then for all 3 initial B frames, ist->frames_decoded will be > 3, and 5 for the last B frame. So, what your patched log tells you is > the total number of previous _successful_ decodes. Presumably, you wish >

Re: [FFmpeg-devel] [PATCH] Display decoded frame number on error

2019-07-12 Thread Nicolas George
Remi Achard (12019-07-12): > So I'm not supposed to use %llu as is the case here (sorry if I didn't > understood your point) ? Exactly, you need to use the format string for uint64_t, PRIu64 from memory; check in the code base and standard. > The use case would be to improve error diagnostic,

Re: [FFmpeg-devel] [PATCH] Display decoded frame number on error

2019-07-12 Thread Remi Achard
> Rémi Achard (12019-07-12): > > --- > > fftools/ffmpeg.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c > > index 01f04103cf..aaeabe512b 100644 > > --- a/fftools/ffmpeg.c > > +++ b/fftools/ffmpeg.c > > @@ -2107,7 +2107,7 @@

Re: [FFmpeg-devel] [PATCH] Display decoded frame number on error

2019-07-12 Thread Nicolas George
Rémi Achard (12019-07-12): > --- > fftools/ffmpeg.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c > index 01f04103cf..aaeabe512b 100644 > --- a/fftools/ffmpeg.c > +++ b/fftools/ffmpeg.c > @@ -2107,7 +2107,7 @@ static void

Re: [FFmpeg-devel] [PATCH] Display decoded frame number on error

2019-07-12 Thread Gyan
On 12-07-2019 02:54 PM, Remi Achard wrote: Thanks for the review, I updated the patch to show the correct frame number as suggested (sorry for the double post I managed to screw up my git send-email). This is not what I meant. Let's call good frames G and bad frames B. If your input is    G

[FFmpeg-devel] [PATCH] libavfilter: add photosensitivity filter

2019-07-12 Thread Paul B Mahol
From: Vladimir Panteleev Signed-off-by: Paul B Mahol --- doc/filters.texi | 20 ++ libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/vf_photosensitivity.c | 336 ++ 4 files changed, 358 insertions(+)

Re: [FFmpeg-devel] [PATCH] Display decoded frame number on error

2019-07-12 Thread Remi Achard
Thanks for the review, I updated the patch to show the correct frame number as suggested (sorry for the double post I managed to screw up my git send-email). Le ven. 12 juil. 2019 à 11:18, Rémi Achard a écrit : > --- > fftools/ffmpeg.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >

[FFmpeg-devel] (no subject)

2019-07-12 Thread Rémi Achard
Good point, thanks for the review, I updated the patch to display the incremented frame number as suggested. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or

[FFmpeg-devel] [PATCH] Display decoded frame number on error

2019-07-12 Thread Rémi Achard
--- fftools/ffmpeg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index 01f04103cf..aaeabe512b 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -2107,7 +2107,7 @@ static void check_decode_result(InputStream *ist, int *got_output,

Re: [FFmpeg-devel] [PATCH 2/3] lavf/webm_chunk: Fix NULL dereference

2019-07-12 Thread Paul B Mahol
On 7/12/19, Andreas Rheinhardt wrote: > Andreas Rheinhardt: >> Andreas Rheinhardt: >>> Andreas Rheinhardt: Andreas Rheinhardt: > The earlier version of the webm_chunk muxer had several bugs: > > 1. If the first packet of an audio stream didn't have a PTS of zero, > then no

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/huffyuv: remove gray8a (the format is listed but not supported by the implementation)

2019-07-12 Thread Paul B Mahol
LGTM On 7/4/19, Michael Niedermayer wrote: > Fixes: null pointer dereference > Fixes: > 15464/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_HYMT_fuzzer-5681391150301184 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed-off-by:

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/pngdec: Check that previous_picture has same w/h/format

2019-07-12 Thread Paul B Mahol
On 7/11/19, Michael Niedermayer wrote: > Fixes: out of array access > Fixes: > 15540/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_APNG_fuzzer-5684905029140480 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed-off-by: Michael

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/huffyuvdec: Check vertical subsampling in hymt

2019-07-12 Thread Paul B Mahol
LGTM On 7/4/19, Michael Niedermayer wrote: > Fixes: out of array access > Fixes: > 15484/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_HYMT_fuzzer-5765377054736384 > Fixes: > 15559/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_HYMT_fuzzer-5710295743332352 > > Found-by: continuous fuzzing

Re: [FFmpeg-devel] [PATCH] Display decoded frame number on error

2019-07-12 Thread Gyan
On 12-07-2019 01:15 PM, Rémi Achard wrote: --- fftools/ffmpeg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index 01f04103cf..fe067007ea 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -2107,7 +2107,7 @@ static void

Re: [FFmpeg-devel] [PATCH] Add support for Display Definition Segment to DVB Subtitle encoder

2019-07-12 Thread Jernej
Here's the fixed patch. I'm not familiar with providing patches via a mailing list. If replying doesn't work, I'll just use a new git send-mail. Signed-off-by: Jernej Fijacko --- libavcodec/dvbsub.c | 19 --- 1 file changed, 16 insertions(+), 3 deletions(-) diff --git

[FFmpeg-devel] [PATCH] Display decoded frame number on error

2019-07-12 Thread Rémi Achard
--- fftools/ffmpeg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index 01f04103cf..fe067007ea 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -2107,7 +2107,7 @@ static void check_decode_result(InputStream *ist, int *got_output,

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg_filter: use -reinit_filter to disable/enable auto scale

2019-07-12 Thread Fu, Linjie
> -Original Message- > From: Li, Zhong > Sent: Friday, July 12, 2019 14:39 > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Cc: Fu, Linjie > Subject: RE: [FFmpeg-devel] [PATCH] fftools/ffmpeg_filter: use -reinit_filter > to disable/enable auto scale > > > From:

[FFmpeg-devel] [PATCH v6] Fix integer parameters size check in SDP fmtp line

2019-07-12 Thread Olivier Maignial
=== PROBLEM === I was trying to record h264 + aac streams from an RTSP server to mp4 file. using this command line: ffmpeg -v verbose -y -i "rtsp:///my_resources" -codec copy -bsf:a aac_adtstoasc test.mp4 FFmpeg then fail to record audio and output this logs: [rtsp @ 0xcda1f0] The

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg_filter: use -reinit_filter to disable/enable auto scale

2019-07-12 Thread Li, Zhong
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Linjie Fu > Sent: Friday, July 12, 2019 9:19 PM > To: ffmpeg-devel@ffmpeg.org > Cc: Fu, Linjie > Subject: [FFmpeg-devel] [PATCH] fftools/ffmpeg_filter: use -reinit_filter to > disable/enable auto scale > > Currently,