[FFmpeg-devel] Problem when remuxing mpeg4 from mp4 container to ts container

Hi! (I recently posted this on -user, but as I fear this might be a bug, I decided to also post here (-devel).) I have a problem where the resulting file from the following commands produce a transport stream file that does not display video. wget http://www.sample-videos.com/video/mp4/480/big_b

Re: [FFmpeg-devel] [PATCH v4] avformat/flvenc: add add_keyframe_index option

2016-11-08 15:13 GMT+08:00 Steven Liu : > Add keyframe index metadata > Used to facilitate seeking; particularly for HTTP pseudo streaming. > 1. read live streaming or file by sequence > 2. if use add_keyframe_index option, add a mark flag at the position, > use to insert new context at the

[FFmpeg-devel] [PATCH v4] avformat/flvenc: add add_keyframe_index option

Add keyframe index metadata Used to facilitate seeking; particularly for HTTP pseudo streaming. 1. read live streaming or file by sequence 2. if use add_keyframe_index option, add a mark flag at the position, use to insert new context at the last step. 3. add the keyframes *offset* and *time

[FFmpeg-devel] [PATCH] Making xmp data process faster

--- libavformat/mov.c | 15 +++ 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index f06de06..341d22d 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -4533,13 +4533,20 @@ static int mov_read_uuid(MOVContext *c, AVIOContext

[FFmpeg-devel] [PATCH] libavcodec/tests: Added test for libavcodec/avpacket.c

Signed-off-by: Thomas Turner --- libavcodec/Makefile | 3 +- libavcodec/tests/avpacket.c | 134 tests/fate/libavcodec.mak | 5 ++ 3 files changed, 141 insertions(+), 1 deletion(-) create mode 100644 libavcodec/tests/avpacket.c diff --gi

[FFmpeg-devel] [PATCH] mov: Read multiple stsd from DV

Signed-off-by: Vittorio Giovara --- Sorry, I can't share the sample or add a fate for this. Please CC. Vittorio libavformat/mov.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index e283034..a2a688b 100644 --- a/libavformat/mov.c +++ b/libavformat/mo

Re: [FFmpeg-devel] [PATCH 2/2] hevc: Support extradata changes

On Mon, Nov 7, 2016 at 6:44 PM, Michael Niedermayer wrote: > the decoder should not change extradata, yes directly reading would > be best probably ok I'll amend. >> >> > also can you add a fate test ? >> >> I have a sample I can share but it's incredibly big for a fate test (85mb). > > :( > > c

[FFmpeg-devel] [PATCH] Making xmp data process faster

Try to jump over all the xmp data in mov->uuid atom, which is faster than read every byte. It's too slow to deal with a mov file contains a big uuid/xmp atom(s), especially in streaming case. 0001-Making-xmp-data-process-faster.patch Description: Binary data _

Re: [FFmpeg-devel] [PATCH 1/2] opus: move all tables to a separate file

On 8 November 2016 at 00:18, Michael Niedermayer wrote: > On Mon, Nov 07, 2016 at 10:39:57PM +, Rostislav Pehlivanov wrote: > > Signed-off-by: Rostislav Pehlivanov > > --- > > libavcodec/Makefile|2 +- > > libavcodec/opus_celt.c | 518 +++--- > > libavcodec/opus_sil

Re: [FFmpeg-devel] [PATCH] aacenc: quit when the audio queue reaches 0 rather than keeping track of empty frames

On 7 November 2016 at 02:14, Rostislav Pehlivanov wrote: > The libopus encoder does the same thing and its better than > keeping track of when the empty flush frames appear. > > Signed-off-by: Rostislav Pehlivanov > --- > libavcodec/aacenc.c | 9 +++-- > libavcodec/aacenc.h | 1 - > 2 files

Re: [FFmpeg-devel] [PATCH v1] add show_demuxers and show_muxers

On Fri, Nov 04, 2016 at 08:20:25PM +0800, Steven Liu wrote: > add -muxers and -demuxers parameters to list the dexmuers and muxers > > Signed-off-by: Steven Liu > --- > cmdutils.c | 66 +++ > cmdutils.h | 14 ++ > cm

Re: [FFmpeg-devel] [PATCH] mov: extract stsd vendor field in metadata.

On Mon, Nov 7, 2016 at 4:30 PM, Zhenni Huang wrote: > Also updates fate ref for rgb24-mkv as the output video will contain the > new metadata field and have different md5sum and file size. > --- > libavformat/mov.c| 25 +++-- > tests/ref/fate/rgb24-mkv | 4 ++-- > 2

[FFmpeg-devel] [PATCH] mov: extract stsd vendor field in metadata.

Also updates fate ref for rgb24-mkv as the output video will contain the new metadata field and have different md5sum and file size. --- libavformat/mov.c| 25 +++-- tests/ref/fate/rgb24-mkv | 4 ++-- 2 files changed, 25 insertions(+), 4 deletions(-) diff --git a/liba

[FFmpeg-devel] [PATCH 2/2] avcodec/hap: add "compressor" option to Hap encoder to disable secondary compression

The secondary compression in Hap is optional, this change exposes that option to the user as some use-cases favour higher bitrate files to reduce workload decoding. Adds "none" or "snappy" as options for "compressor". Selecting "none" disregards "chunks" option: chunking is only of benefit decompre

[FFmpeg-devel] [PATCH 1/2] avcodec/hap: pass texture-compression destination as argument, not in context

This allows a subsequent change to compress directly into the output packet when possible. --- libavcodec/hapenc.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/libavcodec/hapenc.c b/libavcodec/hapenc.c index 076923b..7056b62 100644 --- a/libavcodec/hapenc.c +++ b/libav

Re: [FFmpeg-devel] [PATCH 1/2] opus: move all tables to a separate file

On Mon, Nov 07, 2016 at 10:39:57PM +, Rostislav Pehlivanov wrote: > Signed-off-by: Rostislav Pehlivanov > --- > libavcodec/Makefile|2 +- > libavcodec/opus_celt.c | 518 +++--- > libavcodec/opus_silk.c | 849 +++- > libavcodec/opustab.

Re: [FFmpeg-devel] [PATCH] matroskadec: fix NULL pointer dereference in webm_dash_manifest_read_header

On Tue, Nov 08, 2016 at 12:44:56AM +0100, Andreas Cadhalpun wrote: > The code assumes that s->streams[0] is valid. > > Signed-off-by: Andreas Cadhalpun > --- > libavformat/matroskadec.c | 5 + > 1 file changed, 5 insertions(+) should be ok [...] -- Michael GnuPG fingerprint: 9FF2128B1

Re: [FFmpeg-devel] [PATCH] avformat/matroskadec: fix DiscardPadding element parsing

On Sat, Nov 5, 2016 at 1:50 PM, James Almer wrote: > If the value is negative then it means padding at the start of the packet > instead of at the end. > > Based on a patch by Hendrik Leppkes. > > Signed-off-by: James Almer > --- > libavformat/matroskadec.c | 12 +--- > 1 file changed, 9

Re: [FFmpeg-devel] [PATCH 2/3] lavfi/loudnorm: add an internal libebur128 library

On Fri, 4 Nov 2016, Marton Balint wrote: On Thu, 3 Nov 2016, Hendrik Leppkes wrote: On Mon, Oct 17, 2016 at 5:20 PM, Moritz Barsnick wrote: On Mon, Oct 17, 2016 at 17:09:15 +0200, wm4 wrote: Does this copy parts of libebur128 to FFmpeg? Why? There was a long discussion regarding this pa

Re: [FFmpeg-devel] [PATCH] mpegts: prevent division by zero

On Mon, 7 Nov 2016, Andreas Cadhalpun wrote: Signed-off-by: Andreas Cadhalpun --- libavformat/mpegts.c | 4 1 file changed, 4 insertions(+) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index fad10c6..77d63f2 100644 --- a/libavformat/mpegts.c +++ b/libavformat/mpegts.c @@ -2692

Re: [FFmpeg-devel] [PATCH] mpegts: prevent division by zero

On Mon, Nov 07, 2016 at 11:49:52PM +0100, Andreas Cadhalpun wrote: > Signed-off-by: Andreas Cadhalpun > --- > libavformat/mpegts.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c > index fad10c6..77d63f2 100644 > --- a/libavformat/mpegts.c

[FFmpeg-devel] [PATCH] matroskadec: fix NULL pointer dereference in webm_dash_manifest_read_header

The code assumes that s->streams[0] is valid. Signed-off-by: Andreas Cadhalpun --- libavformat/matroskadec.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c index 32f5e49..130d92e 100644 --- a/libavformat/matroskadec.c +++ b/libavfor

Re: [FFmpeg-devel] [PATCH 2/2] hevc: Support extradata changes

On Mon, Nov 07, 2016 at 04:52:23PM -0500, Vittorio Giovara wrote: > On Sat, Nov 5, 2016 at 9:21 AM, Michael Niedermayer > wrote: > > On Wed, Nov 02, 2016 at 11:48:58AM -0400, Vittorio Giovara wrote: > >> Signed-off-by: Vittorio Giovara > >> --- > >> Please CC. > >> Vittorio > >> > >> libavcodec/

[FFmpeg-devel] [PATCH] mpegts: prevent division by zero

Signed-off-by: Andreas Cadhalpun --- libavformat/mpegts.c | 4 1 file changed, 4 insertions(+) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index fad10c6..77d63f2 100644 --- a/libavformat/mpegts.c +++ b/libavformat/mpegts.c @@ -2692,6 +2692,10 @@ static int mpegts_read_header(AV

Re: [FFmpeg-devel] [libav-devel] [PATCH] mpegpicture: use coded_width/coded_height to allocate frame

On 07.11.2016 22:52, Luca Barbato wrote: > On 07/11/2016 22:32, Andreas Cadhalpun wrote: >> This fixes a heap-buffer-overflow in ff_er_frame_end when decoding mss2 with >> coded_width/coded_height larger than width/height. >> >> Signed-off-by: Andreas Cadhalpun >> --- >> libavcodec/mpegpicture.c

[FFmpeg-devel] [PATCH 2/2] opus: move the entropy decoding functions to opus_rc.c

The intention is to have both encoding and decoding functions in opus_rc.c. Signed-off-by: Rostislav Pehlivanov --- libavcodec/Makefile| 2 +- libavcodec/opus.h | 222 + libavcodec/opus_celt.c | 58 ++--- libavcodec/opus_rc.c

Re: [FFmpeg-devel] [PATCH 1/2] Move av_show_help_children() to avutil/opt

Hi On 11/07/2016 01:34 PM, Andreas Cadhalpun wrote: [...] Isn't the causality the other way around? Because ffserver uses internal API it was decided to remove it. If it gets fixed, there is no point in removing it, or is there? [..] I see no reason to remove it if it works properly. Bests,

Re: [FFmpeg-devel] [PATCH 2/2] hevc: Support extradata changes

On Sat, Nov 5, 2016 at 9:21 AM, Michael Niedermayer wrote: > On Wed, Nov 02, 2016 at 11:48:58AM -0400, Vittorio Giovara wrote: >> Signed-off-by: Vittorio Giovara >> --- >> Please CC. >> Vittorio >> >> libavcodec/hevc.c | 18 ++ >> libavformat/mov.c | 4 >> 2 files changed,

Re: [FFmpeg-devel] [PATCH 1/2] Move av_show_help_children() to avutil/opt

On Mon, Nov 07, 2016 at 10:34:47PM +0100, Andreas Cadhalpun wrote: > On 07.11.2016 20:26, James Almer wrote: > > ffserver is being dropped from the project, as announced in the news page. > > > > Reynaldo however wants to have a working copy somewhere so he's moving it to > > an external repo and

Re: [FFmpeg-devel] [PATCH 1/2] Move av_show_help_children() to avutil/opt

On 11/7/2016 6:34 PM, Andreas Cadhalpun wrote: > On 07.11.2016 20:26, James Almer wrote: >> ffserver is being dropped from the project, as announced in the news page. >> >> Reynaldo however wants to have a working copy somewhere so he's moving it to >> an external repo and for that he's making it w

Re: [FFmpeg-devel] [PATCH 1/2] Move av_show_help_children() to avutil/opt

On 07.11.2016 20:26, James Almer wrote: > ffserver is being dropped from the project, as announced in the news page. > > Reynaldo however wants to have a working copy somewhere so he's moving it to > an external repo and for that he's making it work without internal API, with > some help from Mich

[FFmpeg-devel] [PATCH] mpegpicture: use coded_width/coded_height to allocate frame

This fixes a heap-buffer-overflow in ff_er_frame_end when decoding mss2 with coded_width/coded_height larger than width/height. Signed-off-by: Andreas Cadhalpun --- libavcodec/mpegpicture.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/libavcodec/mpegpicture.c

Re: [FFmpeg-devel] [PATCH 1/2] Move av_show_help_children() to avutil/opt

On 11/7/2016 4:05 PM, Andreas Cadhalpun wrote: > On 04.11.2016 21:16, Michael Niedermayer wrote: >> diff --git a/libavutil/opt.c b/libavutil/opt.c >> index cd16bd1..1b8dae2 100644 >> --- a/libavutil/opt.c >> +++ b/libavutil/opt.c >> @@ -1992,3 +1992,15 @@ int av_opt_serialize(void *obj, int opt_fla

Re: [FFmpeg-devel] [PATCH 1/2] Move av_show_help_children() to avutil/opt

On 04.11.2016 21:16, Michael Niedermayer wrote: > diff --git a/libavutil/opt.c b/libavutil/opt.c > index cd16bd1..1b8dae2 100644 > --- a/libavutil/opt.c > +++ b/libavutil/opt.c > @@ -1992,3 +1992,15 @@ int av_opt_serialize(void *obj, int opt_flags, int > flags, char **buffer, > av_bprint_fina

Re: [FFmpeg-devel] [PATCH] hls: fix leaking avio_opts on hls_read_header error

On 07.11.2016 16:20, Michael Niedermayer wrote: > On Mon, Nov 07, 2016 at 12:11:25AM +0100, Andreas Cadhalpun wrote: >> Use the hls_close function to reduce code duplication. >> >> Signed-off-by: Andreas Cadhalpun >> --- >> libavformat/hls.c | 30 ++ >> 1 file changed,

Re: [FFmpeg-devel] [PATCH] mpegaudio_parser: don't return AVERROR_PATCHWELCOME

On 07.11.2016 10:23, Michael Niedermayer wrote: > On Mon, Nov 07, 2016 at 01:21:01AM +0100, Andreas Cadhalpun wrote: >> The API does not allow returning AVERROR codes. >> >> It triggers an assert in av_parser_parse2. >> >> Signed-off-by: Andreas Cadhalpun >> --- >> libavcodec/mpegaudio_parser.c |

Re: [FFmpeg-devel] [PATCH 3/3 v2] avformat/hls: Add missing error check for avcodec_parameters_copy()

07.11.2016, 01:08, Andreas Cadhalpun kirjoitti: > On 06.11.2016 23:52, Anssi Hannula wrote: >> Signed-off-by: Anssi Hannula >> --- >> >> 07.11.2016, 00:35, Andreas Cadhalpun kirjoitti: >>> On 06.11.2016 22:44, Anssi Hannula wrote: Signed-off-by: Anssi Hannula --- libavformat/hls.c

Re: [FFmpeg-devel] [PATCH v3] avformat/flvenc: add add_keyframe_index option

On Mon, Nov 07, 2016 at 06:02:05PM +0800, Steven Liu wrote: > Add keyframe index metadata > Used to facilitate seeking; particularly for HTTP pseudo streaming. > 1. read live streaming or file by sequence > 2. if use add_keyframe_index option, add a mark flag at the position, > use to insert

Re: [FFmpeg-devel] [PATCH] hls: fix leaking avio_opts on hls_read_header error

On Mon, Nov 07, 2016 at 12:11:25AM +0100, Andreas Cadhalpun wrote: > Use the hls_close function to reduce code duplication. > > Signed-off-by: Andreas Cadhalpun > --- > libavformat/hls.c | 30 ++ > 1 file changed, 14 insertions(+), 16 deletions(-) LGTM but please spl

Re: [FFmpeg-devel] [PATCH v1] add show_demuxers and show_muxers

ping 2016-11-04 20:20 GMT+08:00 Steven Liu : > add -muxers and -demuxers parameters to list the dexmuers and muxers > > Signed-off-by: Steven Liu > --- > cmdutils.c | 66 ++ > + > cmdutils.h | 14 ++ > cmdutils_comm

Re: [FFmpeg-devel] [PATCH] WIP: subtitles in AVFrame

On Thu, Nov 03, 2016 at 01:05:01AM +0100, Marton Balint wrote: [...] > > So long story short: AVSubtitle → AVFrame, you have to use the new M:N > > API, and it's integrated into lavfi. > > > > Now a bit longer: the AVFrame->extended_data are > > AVFrameSubtitleRectangle (declared in lavu/frame). >

Re: [FFmpeg-devel] [PATCH 0/2] Move show_*() code from cmdutils to libs

On Fri, Nov 04, 2016 at 09:16:34PM +0100, Michael Niedermayer wrote: > > Printing the available codecs, decoders, encoders, bitstream filters and their > options and capabilities is usefull for Apps in general and not just ffmpeg. > > This patch-set is also required for moving ffserver into a sep

[FFmpeg-devel] Broken libnpp (was working with git snapshot 0779396)

Hi All, I was trying libnpp on Big Bunny video by below command: ./ffmpeg -loglevel debug -y -hwaccel cuvid -c:v h264_cuvid -vsync 0 -i /root/root/bunny.mp4 -vf scale_npp=1920:1072 -vcodec h264_nvenc /tmp/tmp0.264 -vf scale_npp=1280:720 -vcodec h264_nvenc /tmp/tmp1.264 This command is working wi

[FFmpeg-devel] [PATCH v3] avformat/flvenc: add add_keyframe_index option

Add keyframe index metadata Used to facilitate seeking; particularly for HTTP pseudo streaming. 1. read live streaming or file by sequence 2. if use add_keyframe_index option, add a mark flag at the position, use to insert new context at the last step. 3. add the keyframes *offset* and *time

Re: [FFmpeg-devel] [PATCH] mpegaudio_parser: don't return AVERROR_PATCHWELCOME

On Mon, Nov 07, 2016 at 01:21:01AM +0100, Andreas Cadhalpun wrote: > The API does not allow returning AVERROR codes. > > It triggers an assert in av_parser_parse2. > > Signed-off-by: Andreas Cadhalpun > --- > libavcodec/mpegaudio_parser.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)