Re: [FFmpeg-devel] [PATCH] avutil/opencl: don't include config.h

2015-01-07 Thread Wei Gao
2015-01-08 10:10 GMT+08:00 James Almer : > On 07/01/15 10:55 PM, Wei Gao wrote: > > 2015-01-08 8:42 GMT+08:00 Michael Niedermayer : > > > >> On Wed, Jan 07, 2015 at 05:44:41PM -0300, James Almer wrote: > >>> On 06/01/15 11:27 PM, Michael Niedermayer wrote: > On Tue, Jan 06, 2015 at 01:56:01AM

[FFmpeg-devel] Unable to transcode audio from broadcast ts stream

2015-01-07 Thread William Juwono
Hi, I am having problem transcoding the audio of the following file. This is captured from broadcast ts stream.http://www.datafilehost.com/d/cb3d03be If I use "acodec copy", the resulting audio played fine, however, if I transcode to other formats, ie: aac, mp3, it resulted in chopped audio.Can

[FFmpeg-devel] [PATCH] ffmpeg: Add bitrate value under -progress action

2015-01-07 Thread Michael Niedermayer
From: raxon1s --- ffmpeg.c | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index ddf4272..28c76c5 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -1526,10 +1526,15 @@ static void print_report(int is_last_report, int64_t timer_start, int64_t cur_ti

Re: [FFmpeg-devel] [PATCH] avutil/opencl: don't include config.h

2015-01-07 Thread James Almer
On 07/01/15 10:55 PM, Wei Gao wrote: > 2015-01-08 8:42 GMT+08:00 Michael Niedermayer : > >> On Wed, Jan 07, 2015 at 05:44:41PM -0300, James Almer wrote: >>> On 06/01/15 11:27 PM, Michael Niedermayer wrote: On Tue, Jan 06, 2015 at 01:56:01AM -0300, James Almer wrote: > It's not an installe

Re: [FFmpeg-devel] [PATCH] avutil/opencl: don't include config.h

2015-01-07 Thread Wei Gao
2015-01-08 8:42 GMT+08:00 Michael Niedermayer : > On Wed, Jan 07, 2015 at 05:44:41PM -0300, James Almer wrote: > > On 06/01/15 11:27 PM, Michael Niedermayer wrote: > > > On Tue, Jan 06, 2015 at 01:56:01AM -0300, James Almer wrote: > > >> It's not an installed header. Use libavutil/avconfig.h inste

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/dvdsubdec: fix accessing dangling pointers

2015-01-07 Thread Michael Niedermayer
On Wed, Jan 07, 2015 at 11:57:51PM +0100, wm4 wrote: > dvdsub_decode() can call append_to_cached_buf() 2 times, the second time > with ctx->buf as argument. If the second append_to_cached_buf() reallocs > ctx->buf, the argument will be a pointer to the previous, freed block. > This can cause invali

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/dvdsubdec: error on bitmaps with size 0

2015-01-07 Thread Michael Niedermayer
On Wed, Jan 07, 2015 at 11:57:50PM +0100, wm4 wrote: > Attemtping to decode them could lead to invalid writes with some fuzzed > samples. > --- > libavcodec/dvdsubdec.c | 3 +++ > 1 file changed, 3 insertions(+) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611

Re: [FFmpeg-devel] [PATCH] avutil/opencl: don't include config.h

2015-01-07 Thread Michael Niedermayer
On Wed, Jan 07, 2015 at 05:44:41PM -0300, James Almer wrote: > On 06/01/15 11:27 PM, Michael Niedermayer wrote: > > On Tue, Jan 06, 2015 at 01:56:01AM -0300, James Almer wrote: > >> It's not an installed header. Use libavutil/avconfig.h instead. > > > > assuming noone objects to this "hack" ... ug

[FFmpeg-devel] [PATCH 1/2] avcodec/dvdsubdec: error on bitmaps with size 0

2015-01-07 Thread wm4
Attemtping to decode them could lead to invalid writes with some fuzzed samples. --- libavcodec/dvdsubdec.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/dvdsubdec.c b/libavcodec/dvdsubdec.c index 2423cad..1cc34ea 100644 --- a/libavcodec/dvdsubdec.c +++ b/libavcodec/dvdsubdec.c

[FFmpeg-devel] [PATCH 2/2] avcodec/dvdsubdec: fix accessing dangling pointers

2015-01-07 Thread wm4
dvdsub_decode() can call append_to_cached_buf() 2 times, the second time with ctx->buf as argument. If the second append_to_cached_buf() reallocs ctx->buf, the argument will be a pointer to the previous, freed block. This can cause invalid reads at least with some fuzzed files - and possibly with v

Re: [FFmpeg-devel] [PATCH] avutil/opencl: don't include config.h

2015-01-07 Thread James Almer
On 06/01/15 11:27 PM, Michael Niedermayer wrote: > On Tue, Jan 06, 2015 at 01:56:01AM -0300, James Almer wrote: >> It's not an installed header. Use libavutil/avconfig.h instead. > > assuming noone objects to this "hack" ... ugly but it should fix the > build so ok Yes, i agree it's kinda ugly. W

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: added HLS encryption

2015-01-07 Thread Michael Niedermayer
On Wed, Jan 07, 2015 at 08:59:11AM -0600, Christian Suloway wrote: > Added HLS encryption with -hls_key_info_file option. The > first line of key_info_file specifies the key URI for the playlist. The > second line specifies the path to the file containing the encryption > key. An optional third li

Re: [FFmpeg-devel] [PATCH 1/4] Correctly set Display Aspect Ratio in nvenc.

2015-01-07 Thread Mark Himsley
On 07/01/2015 18:46, Philip Langdale wrote: > On Wed, 7 Jan 2015 12:13:25 +0100 > Nicolas George wrote: > >> Can you explain the 1024? >> >> Anyway, I would suggest to use lavu's rational functions: >> >> AVRational dar = av_mul_q(avctx->sample_aspect_ratio, >> a

Re: [FFmpeg-devel] [PATCH 1/4] Correctly set Display Aspect Ratio in nvenc.

2015-01-07 Thread Philip Langdale
On Wed, 7 Jan 2015 10:46:47 -0800 Philip Langdale wrote: > > Honestly, I don't understand what the encoder is doing at this point. > The 1.02 seems to be the 'correct' scale factor for a PAL DVD, but is > incorrect for anything else. I used 1024 to avoid the floating point > problem, but you are

Re: [FFmpeg-devel] [PATCH 1/4] Correctly set Display Aspect Ratio in nvenc.

2015-01-07 Thread Philip Langdale
On Wed, 7 Jan 2015 12:13:25 +0100 Nicolas George wrote: > Can you explain the 1024? > > Anyway, I would suggest to use lavu's rational functions: > > AVRational dar = av_mul_q(avctx->sample_aspect_ratio, > av_make_q(avctx->width, avctx->height)); > av_reduc

[FFmpeg-devel] [PATCH v2] libavformat/mxfdec.c: support demuxing opatom audio without index

2015-01-07 Thread Mark Reid
Hi, This approach is much simpler then my previous attempt. This version creates a IndexTableSegment and everything else can continue as normal. --- libavformat/mxfdec.c | 56 1 file changed, 56 insertions(+) diff --git a/libavformat/mxfdec.

Re: [FFmpeg-devel] [PATCH 5/5] Changelog: mention edit list support

2015-01-07 Thread Michael Niedermayer
On Tue, Jan 06, 2015 at 06:10:01PM +0100, Clément Bœsch wrote: > From: Clément Bœsch > > --- > Changelog | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Changelog b/Changelog > index 4575e8b..43998cc 100644 > --- a/Changelog > +++ b/Changelog > @@ -11,6 +11,7 @@ version : > - tblend f

Re: [FFmpeg-devel] [PATCH] lavfi: Port mp=pp7 to libavfilter

2015-01-07 Thread Michael Niedermayer
On Wed, Jan 07, 2015 at 03:42:16PM +0100, Stefano Sabatini wrote: > On date Wednesday 2015-01-07 19:14:49 +0530, arwa arif encoded: > > > > > > > > > Keep in mind that the difference may be due to the QP information > > > missing in mp=pp7. So you should focus on that (try to reset/disable > > > qp

[FFmpeg-devel] [PATCH] avformat/hlsenc: added HLS encryption

2015-01-07 Thread Christian Suloway
Added HLS encryption with -hls_key_info_file option. The first line of key_info_file specifies the key URI for the playlist. The second line specifies the path to the file containing the encryption key. An optional third line specifies an IV to use instead of the segment number. Changes to key_inf

Re: [FFmpeg-devel] [PATCH] lavfi: Port mp=pp7 to libavfilter

2015-01-07 Thread Stefano Sabatini
On date Wednesday 2015-01-07 19:14:49 +0530, arwa arif encoded: > > > > > > Keep in mind that the difference may be due to the QP information > > missing in mp=pp7. So you should focus on that (try to reset/disable > > qp processing in pp7 and see if it works the same as mp=pp7). > > > > For qp=0

Re: [FFmpeg-devel] [PATCH] lavfi: Port mp=pp7 to libavfilter

2015-01-07 Thread arwa arif
> > > Keep in mind that the difference may be due to the QP information > missing in mp=pp7. So you should focus on that (try to reset/disable > qp processing in pp7 and see if it works the same as mp=pp7). > For qp=0, the results are bitexact. I am not able to figure out how do I take care of the

Re: [FFmpeg-devel] [PATCH 1/4] Correctly set Display Aspect Ratio in nvenc.

2015-01-07 Thread Nicolas George
Le sextidi 16 nivôse, an CCXXIII, Philip Langdale a écrit : > The encoder writes the display aspect ratio into the output frames, which a > good player will respect, so let's make sure it's correct. > > The 1.02 scale factor is black magic, but produces correct results. I don't > know what nvenc

[FFmpeg-devel] [PATCH] avcodec/nvenc: fix dts>pts when b frame >= 1

2015-01-07 Thread Agatha Hu
When B frames >= 1, there'll be a invalid dts/pts warning during encoding like: [mp4 @ 0034cba0] Invalid DTS: 1024 PTS: 512 in output stream 0:0, replacing by guess [mp4 @ 0034cba0] Invalid DTS: 1536 PTS: 1024 in output stream 0:0, replacing by guess Set a -1 offset to dts to