[FFmpeg-devel] GSoC and Outreachy

2015-03-02 Thread Michael Niedermayer
Hi all FFmpeg has been accepted in GSoC this year. If you want to mentor or are considering to mentor then please subscribe to https://ffmpeg.org/mailman/listinfo/ffmpeg-mentors (this list is ONLY for discussions that need to be private, when in doubt use the public ffmpeg-devel list) The

Re: [FFmpeg-devel] [PATCH] avcodec/proresenc_kostya: fix coded_frame handling

2015-03-02 Thread Clément Bœsch
On Mon, Mar 02, 2015 at 10:23:54PM +0100, Michael Niedermayer wrote: On Mon, Mar 02, 2015 at 09:20:23PM +0100, Clément Bœsch wrote: --- libavcodec/proresenc_kostya.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) LGTM Applied, thanks. -- Clément B.

Re: [FFmpeg-devel] GSoC and Outreachy

2015-03-02 Thread Thierry Foucu
On Mon, Mar 2, 2015 at 1:39 PM, Michael Niedermayer michae...@gmx.at wrote: Hi all FFmpeg has been accepted in GSoC this year. This is a GREAT news.. Congratulation to the people working on it If you want to mentor or are considering to mentor then please subscribe to

Re: [FFmpeg-devel] [PATCH] avcodec/webp: validate the distance prefix code

2015-03-02 Thread Michael Niedermayer
On Mon, Mar 02, 2015 at 08:58:45PM +0100, Andreas Cadhalpun wrote: Hi, according to the WebP Lossless Bitstream Specification [1] the highest allowed value for the prefix code is 39. Attached patch adds a check for this to avoid crashes decoding broken files. Best regards, Andreas

Re: [FFmpeg-devel] [PATCH] avcodec/proresenc_kostya: fix coded_frame handling

2015-03-02 Thread Michael Niedermayer
On Mon, Mar 02, 2015 at 09:20:23PM +0100, Clément Bœsch wrote: --- libavcodec/proresenc_kostya.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) LGTM [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB I know you won't believe me, but the

Re: [FFmpeg-devel] [PATCH] acvodec/lipopenjpeg: Improve XYZ color space detection

2015-03-02 Thread Michael Bradshaw
On Mon, Mar 2, 2015 at 4:48 PM, Vilius Grigaliūnas vilius.grigaliu...@gmail.com wrote: The reason for this change is that the native jpeg2000 decoder does not yet support 4K digital cinema files (#2586). The workaround for that is to use libopenjpeg decoder, which sort of works but

[FFmpeg-devel] [PATCH v3 1/2] libavformat/avio: added avio_put_str16be

2015-03-02 Thread Mark Reid
--- doc/APIchanges| 3 +++ libavformat/avio.h| 6 ++ libavformat/aviobuf.c | 23 +++ libavformat/version.h | 4 ++-- 4 files changed, 30 insertions(+), 6 deletions(-) diff --git a/doc/APIchanges b/doc/APIchanges index 5fdfc82..6875aed 100644 ---

[FFmpeg-devel] [PATCH v3 2/2] libavformat/mxfenc: write package name metadata

2015-03-02 Thread Mark Reid
--- libavformat/mxfenc.c | 88 +-- tests/ref/lavf/mxf| 6 ++-- tests/ref/lavf/mxf_d10| 2 +- tests/ref/lavf/mxf_opatom | 2 +- 4 files changed, 82 insertions(+), 16 deletions(-) diff --git a/libavformat/mxfenc.c

[FFmpeg-devel] [PATCH v3 0/2] libavformat/mxfenc: write package name metadata

2015-03-02 Thread Mark Reid
changes since v2 * simplified put_str16 macro * added entry to doc/APIchanges Mark Reid (2): libavformat/avio: added avio_put_str16be libavformat/mxfenc: write package name metadata doc/APIchanges| 3 ++ libavformat/avio.h| 6 libavformat/aviobuf.c | 23

Re: [FFmpeg-devel] [PATCH] Port FFT domain filter.

2015-03-02 Thread arwa arif
I tried to change the name to fft, but it is conflicting with libavcodec/fft. So, I am unable to change that. I have updated the patch, adding support to coloured images. From 7b479d2236067e5599f6e4670f319be4a643 Mon Sep 17 00:00:00 2001 From: Arwa Arif arwaarif1...@gmail.com Date: Tue, 24 Feb

Re: [FFmpeg-devel] [PATCH 1/3] avcodec: use av_frame_free() for coded_frame

2015-03-02 Thread Paul B Mahol
On 2/28/15, Clement Boesch u...@pkh.me wrote: On Sat, Feb 28, 2015 at 10:38:32AM +0100, Clement Boesch wrote: --- libavcodec/avuienc.c| 2 +- libavcodec/libopenjpegenc.c | 4 ++-- libavcodec/libtheoraenc.c | 2 +- libavcodec/libutvideoenc.cpp| 2 +-

Re: [FFmpeg-devel] [PATCH] configure: autodetect opencl

2015-03-02 Thread wm4
On Mon, 2 Mar 2015 07:29:11 + Gupta, Maneesh maneesh.gu...@amd.com wrote: Hi, Similar to the way SDL is being autodetected, the attached patch proposes to autodetect OpenCL. SDL shouldn't be autodetected at all, since it's not a system lib, and often causes all sorts of problems even

Re: [FFmpeg-devel] [PATCH 2/3] avfilter/lavfutils: use av_frame_free for freeing an AVFrame

2015-03-02 Thread Clément Bœsch
On Sat, Feb 28, 2015 at 11:07:23AM +, Paul B Mahol wrote: On 2/28/15, Clement Boesch u...@pkh.me wrote: --- libavfilter/lavfutils.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/lavfutils.c b/libavfilter/lavfutils.c index 80310d2..75e68a7 100644

Re: [FFmpeg-devel] [PATCH 1/3] avcodec: use av_frame_free() for coded_frame

2015-03-02 Thread Clément Bœsch
On Mon, Mar 02, 2015 at 10:01:57AM +0100, Clément Bœsch wrote: On Mon, Mar 02, 2015 at 08:31:16AM +, Paul B Mahol wrote: On 2/28/15, Clement Boesch u...@pkh.me wrote: On Sat, Feb 28, 2015 at 10:38:32AM +0100, Clement Boesch wrote: --- libavcodec/avuienc.c| 2 +-

Re: [FFmpeg-devel] [PATCH] avformat/dashenc: Update codec_str on extradata_size change

2015-03-02 Thread Michael Niedermayer
On Sun, Mar 01, 2015 at 10:59:55PM +0100, Timo Rothenpieler wrote: --- libavformat/dashenc.c | 24 +--- 1 file changed, 21 insertions(+), 3 deletions(-) split and applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB During

Re: [FFmpeg-devel] [PATCH 3/3] avfilter/vf_tblend: use av_frame_free for freeing an AVFrame

2015-03-02 Thread Clément Bœsch
On Sat, Feb 28, 2015 at 11:06:12AM +, Paul B Mahol wrote: On 2/28/15, Clement Boesch u...@pkh.me wrote: --- libavfilter/vf_blend.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/vf_blend.c b/libavfilter/vf_blend.c index a29cabc..d232457 100644

Re: [FFmpeg-devel] [RFC][PATCH] ffplay: factorize subtitle rendering code to a private filter

2015-03-02 Thread Clément Bœsch
On Tue, Feb 17, 2015 at 03:27:01AM +0100, Marton Balint wrote: On Sat, 14 Feb 2015, Clément Bœsch wrote: On Sat, Feb 14, 2015 at 02:08:15AM +0100, Marton Balint wrote: Signed-off-by: Marton Balint c...@passwd.hu --- Makefile | 1 + doc/ffplay.texi

[FFmpeg-devel] [PATCH] ffplay: Switch to show waves mode if allocation/init of RDFT fails

2015-03-02 Thread Michael Niedermayer
Fixes null pointer dereference Found-by: Paweł pantrom...@gmail.com Signed-off-by: Michael Niedermayer michae...@gmx.at --- ffplay.c |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/ffplay.c b/ffplay.c index 8140dd9..94c7ed4 100644 --- a/ffplay.c +++ b/ffplay.c @@

Re: [FFmpeg-devel] [PATCH] configure: autodetect opencl

2015-03-02 Thread Clément Bœsch
On Mon, Mar 02, 2015 at 08:49:08AM +, Carl Eugen Hoyos wrote: Gupta, Maneesh Maneesh.Gupta at amd.com writes: Similar to the way SDL is being autodetected, the attached patch proposes to autodetect OpenCL. This cannot be done since the API is defined as being not stable. Making

Re: [FFmpeg-devel] [PATCH] configure: autodetect opencl

2015-03-02 Thread Carl Eugen Hoyos
Gupta, Maneesh Maneesh.Gupta at amd.com writes: Similar to the way SDL is being autodetected, the attached patch proposes to autodetect OpenCL. This cannot be done since the API is defined as being not stable. Carl Eugen ___ ffmpeg-devel mailing

Re: [FFmpeg-devel] [PATCH 1/3] avcodec: use av_frame_free() for coded_frame

2015-03-02 Thread Clément Bœsch
On Mon, Mar 02, 2015 at 08:31:16AM +, Paul B Mahol wrote: On 2/28/15, Clement Boesch u...@pkh.me wrote: On Sat, Feb 28, 2015 at 10:38:32AM +0100, Clement Boesch wrote: --- libavcodec/avuienc.c| 2 +- libavcodec/libopenjpegenc.c | 4 ++-- libavcodec/libtheoraenc.c

Re: [FFmpeg-devel] Gamma function (was Re: [PATCH] lavc/pngenc: Support writing colorspace tags.)

2015-03-02 Thread Kevin Wheatley
On Mon, Mar 2, 2015 at 12:17 PM, Michael Niedermayer michae...@gmx.at wrote: the various colorspace options should pass from decoder over the video filter chain to the encoder and then muxer its best to change them from the command line through a video filter, this also ensures that the value

Re: [FFmpeg-devel] Gamma function (was Re: [PATCH] lavc/pngenc: Support writing colorspace tags.)

2015-03-02 Thread Kevin Wheatley
Something like this - note this adds no tests, but fate still passes for me. Kevin From db02ae26c3c4278da4ed328e767bab98271da51e Mon Sep 17 00:00:00 2001 From: Kevin Wheatley kevin.j.wheat...@gmail.com Date: Mon, 2 Mar 2015 12:50:53 + Subject: [PATCH] Add 'gama' atom for .mov only, reuses

[FFmpeg-devel] Gamma function (was Re: [PATCH] lavc/pngenc: Support writing colorspace tags.)

2015-03-02 Thread Kevin Wheatley
On Sat, Feb 28, 2015 at 1:50 AM, Niklas Haas g...@nand.wakku.to wrote: +static int png_get_gama(enum AVColorTransferCharacteristic trc, uint8_t *buf) +{ +double gamma; +switch (trc) { +case AVCOL_TRC_BT709: +case AVCOL_TRC_SMPTE170M: +case

Re: [FFmpeg-devel] [PATCH] configure: autodetect opencl

2015-03-02 Thread Gupta, Maneesh
-Original Message- From: ffmpeg-devel-boun...@ffmpeg.org [mailto:ffmpeg-devel- boun...@ffmpeg.org] On Behalf Of Michael Niedermayer Sent: Monday, March 02, 2015 4:58 PM To: FFmpeg development discussions and patches Subject: Re: [FFmpeg-devel] [PATCH] configure: autodetect opencl

Re: [FFmpeg-devel] [PATCH] aac: smal fix in aacpsy pe reduction step

2015-03-02 Thread Claudio Freire
On Mon, Mar 2, 2015 at 5:50 AM, Carl Eugen Hoyos ceho...@ag.or.at wrote: Claudio Freire klaussfreire at gmail.com writes: testing results: http://pastebin.com/051BXYTy If the testing results are useful please add them to the commit message, please do not use links to pastebin in the commit

Re: [FFmpeg-devel] [PATCH 1/3] avcodec: use av_frame_free() for coded_frame

2015-03-02 Thread Nicolas George
Le duodi 12 ventôse, an CCXXIII, Clement Boesch a écrit : I was confused by the output of FATE, it's not in r210, it's in prores: *avctx-coded_frame = *pic; This code is invalid: if a future version of lavu adds fields at the end of AVFrame, they would not be copied. (I hoped we

Re: [FFmpeg-devel] [PATCH] aac: smal fix in aacpsy pe reduction step

2015-03-02 Thread Michael Niedermayer
On Mon, Mar 02, 2015 at 08:47:21AM -0300, Claudio Freire wrote: On Mon, Mar 2, 2015 at 5:50 AM, Carl Eugen Hoyos ceho...@ag.or.at wrote: Claudio Freire klaussfreire at gmail.com writes: testing results: http://pastebin.com/051BXYTy If the testing results are useful please add them to

Re: [FFmpeg-devel] [PATCH] configure: autodetect opencl

2015-03-02 Thread Nicolas George
Le duodi 12 ventôse, an CCXXIII, Gupta, Maneesh a écrit : I see a lot of modules that are autodetected. And not all of them are system libs. The consensus is this: A. A library that is redundant with a native implementation of a FFmpeg target feature (target = implementing multimedia codecs

Re: [FFmpeg-devel] [PATCH] aac: smal fix in aacpsy pe reduction step

2015-03-02 Thread Claudio Freire
Great, picking next patch then. On Mon, Mar 2, 2015 at 9:48 AM, Michael Niedermayer michae...@gmx.at wrote: On Mon, Mar 02, 2015 at 08:47:21AM -0300, Claudio Freire wrote: On Mon, Mar 2, 2015 at 5:50 AM, Carl Eugen Hoyos ceho...@ag.or.at wrote: Claudio Freire klaussfreire at gmail.com writes:

Re: [FFmpeg-devel] Gamma function (was Re: [PATCH] lavc/pngenc: Support writing colorspace tags.)

2015-03-02 Thread Michael Niedermayer
On Mon, Mar 02, 2015 at 11:00:43AM +, Kevin Wheatley wrote: On Sat, Feb 28, 2015 at 1:50 AM, Niklas Haas g...@nand.wakku.to wrote: +static int png_get_gama(enum AVColorTransferCharacteristic trc, uint8_t *buf) +{ +double gamma; +switch (trc) { +case

Re: [FFmpeg-devel] [PATCH] x86/hevc_sao: use unaligned movs for sao_{band, filter} with width 8

2015-03-02 Thread Christophe Gisquet
Hi, 2015-03-01 22:07 GMT+01:00 James Almer jamr...@gmail.com: @@ -537,7 +537,7 @@ align 16 HEVC_SAO_EDGE_FILTER_COMPUTE_10 CLIPW m2, m0, [pw_mask %+ %1] -mova [dstq], m2 +movu [dstq], m2 %endif %assign i 0 -- 2.3.0 Not really

Re: [FFmpeg-devel] [PATCH 1/3] avcodec: use av_frame_free() for coded_frame

2015-03-02 Thread Paul B Mahol
On 3/2/15, Clement Boesch u...@pkh.me wrote: On Mon, Mar 02, 2015 at 08:31:16AM +, Paul B Mahol wrote: On 2/28/15, Clement Boesch u...@pkh.me wrote: On Sat, Feb 28, 2015 at 10:38:32AM +0100, Clement Boesch wrote: --- libavcodec/avuienc.c| 2 +-

Re: [FFmpeg-devel] [PATCH] aac: smal fix in aacpsy pe reduction step

2015-03-02 Thread Carl Eugen Hoyos
Claudio Freire klaussfreire at gmail.com writes: testing results: http://pastebin.com/051BXYTy If the testing results are useful please add them to the commit message, please do not use links to pastebin in the commit message. Thank you, Carl Eugen

Re: [FFmpeg-devel] [PATCH] tests/fate: Add S302M test

2015-03-02 Thread Michael Niedermayer
On Mon, Mar 02, 2015 at 08:50:08AM +0100, Reimar Döffinger wrote: On 02.03.2015, at 00:41, Michael Niedermayer michae...@gmx.at wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- tests/fate/acodec.mak |6 ++ tests/ref/acodec/s302m |4 2 files changed, 10

Re: [FFmpeg-devel] [PATCH 1/2] avformat/rtpdec_mpa_robust: fix commit 96084251e57d1738fde02a2b0d37ca609d9efd71

2015-03-02 Thread Michael Niedermayer
On Mon, Mar 02, 2015 at 11:00:16AM +0100, Gilles Chanteperdrix wrote: Commit 96084251e57d1738fde02a2b0d37ca609d9efd71 introduced a change in the parser implementation which broke it. Restore the original implementation. Signed-off-by: Gilles Chanteperdrix gilles.chanteperd...@xenomai.org

[FFmpeg-devel] [PATCH 1/2] avformat/rtpdec_mpa_robust: fix commit 96084251e57d1738fde02a2b0d37ca609d9efd71

2015-03-02 Thread Gilles Chanteperdrix
Commit 96084251e57d1738fde02a2b0d37ca609d9efd71 introduced a change in the parser implementation which broke it. Restore the original implementation. Signed-off-by: Gilles Chanteperdrix gilles.chanteperd...@xenomai.org --- libavformat/rtpdec_mpa_robust.c | 2 +- 1 file changed, 1 insertion(+), 1

[FFmpeg-devel] [PATCH 2/2] MAINTAINERS: add myself as rtpdec_mpa_robust maintainer

2015-03-02 Thread Gilles Chanteperdrix
Signed-off-by: Gilles Chanteperdrix gilles.chanteperd...@xenomai.org --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 0cd8c26..007540c 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -471,6 +471,7 @@ Muxers/Demuxers: rtpdec_dv.*

Re: [FFmpeg-devel] [PATCH] configure: autodetect opencl

2015-03-02 Thread Gupta, Maneesh
-Original Message- From: ffmpeg-devel-boun...@ffmpeg.org [mailto:ffmpeg-devel- boun...@ffmpeg.org] On Behalf Of wm4 Sent: Monday, March 02, 2015 2:26 PM To: ffmpeg-devel@ffmpeg.org Subject: Re: [FFmpeg-devel] [PATCH] configure: autodetect opencl On Mon, 2 Mar 2015 07:29:11 +

Re: [FFmpeg-devel] [PATCH] configure: autodetect opencl

2015-03-02 Thread Michael Niedermayer
On Mon, Mar 02, 2015 at 10:59:10AM +, Carl Eugen Hoyos wrote: Gupta, Maneesh Maneesh.Gupta at amd.com writes: Similar to the way SDL is being autodetected, the attached patch proposes to autodetect OpenCL. This cannot be done since the API is defined as being not stable.

Re: [FFmpeg-devel] [PATCH] mxfenc: Adjust product name when opatom muxer is used

2015-03-02 Thread Robert Krüger
On Sun, Mar 1, 2015 at 10:43 PM, Mark Reid mindm...@gmail.com wrote: On Sun, Mar 1, 2015 at 4:06 AM, Robert Krüger krue...@lesspain.de wrote: Currently the product name that ends up in mxf files muxed using the new op atom muxer is OP1A muxer which is misleading. Attached patch changes

Re: [FFmpeg-devel] [PATCH] configure: autodetect opencl

2015-03-02 Thread Gupta, Maneesh
-Original Message- From: ffmpeg-devel-boun...@ffmpeg.org [mailto:ffmpeg-devel- boun...@ffmpeg.org] On Behalf Of Carl Eugen Hoyos Sent: Monday, March 02, 2015 2:19 PM To: ffmpeg-devel@ffmpeg.org Subject: Re: [FFmpeg-devel] [PATCH] configure: autodetect opencl Gupta, Maneesh

Re: [FFmpeg-devel] [PATCH] configure: autodetect opencl

2015-03-02 Thread Carl Eugen Hoyos
Gupta, Maneesh Maneesh.Gupta at amd.com writes: Similar to the way SDL is being autodetected, the attached patch proposes to autodetect OpenCL. This cannot be done since the API is defined as being not stable. Sorry. I am not sure what you mean by this. Do you mean OpenCL API

Re: [FFmpeg-devel] [libav-devel] [PATCH] avformat/flvenc: check that the codec_tag fits in one byte

2015-03-02 Thread Andreas Cadhalpun
On 01.03.2015 21:25, Luca Barbato wrote: On 01/03/15 21:22, Diego Biurrun wrote: On Sat, Feb 28, 2015 at 09:12:57PM +0100, Andreas Cadhalpun wrote: From 850352d0e1c262b434094fb19787bbd5b6bf6c57 Mon Sep 17 00:00:00 2001 From: Andreas Cadhalpun andreas.cadhal...@googlemail.com Date: Sat, 28 Feb

Re: [FFmpeg-devel] [libav-devel] [PATCH] avformat/rmenc: limit packet size

2015-03-02 Thread Andreas Cadhalpun
On 02.03.2015 16:21, Luca Barbato wrote: On 02/03/15 16:06, Andreas Cadhalpun wrote: Yes, the rm format can contain larger frames split into slices and packed in multiple packets. But currently the rv10/rv20 encoders always produce a whole frame and the rm muxer always produces one packet per

Re: [FFmpeg-devel] [libav-devel] [PATCH] avformat/rmenc: limit packet size

2015-03-02 Thread Andreas Cadhalpun
On 01.03.2015 21:18, Diego Biurrun wrote: On Sat, Feb 28, 2015 at 11:40:30PM +0100, Andreas Cadhalpun wrote: From 4bb0687f07713787f83dcd90214ad32b91c0f91f Mon Sep 17 00:00:00 2001 From: Andreas Cadhalpun andreas.cadhal...@googlemail.com Date: Sat, 28 Feb 2015 23:11:50 +0100 Subject: [PATCH]

Re: [FFmpeg-devel] [PATCH 1/2] libavformat/avio: added avio_put_str16be

2015-03-02 Thread Clément Bœsch
On Sun, Mar 01, 2015 at 01:30:54PM -0800, Mark Reid wrote: --- libavformat/avio.h| 6 ++ libavformat/aviobuf.c | 50 +++--- libavformat/version.h | 2 +- 3 files changed, 34 insertions(+), 24 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH] avcodec/pngenc: cast double to integer type to avoid problems with some AV_WB32() implementations

2015-03-02 Thread Clément Bœsch
On Sun, Mar 01, 2015 at 08:07:46PM -0500, Ronald S. Bultje wrote: Hi, On Sun, Mar 1, 2015 at 7:38 PM, Michael Niedermayer michae...@gmx.at wrote: alternatively lrint(f)() could be used or various other options Signed-off-by: Michael Niedermayer michae...@gmx.at ---

Re: [FFmpeg-devel] [PATCH] avcodec/pngenc: cast double to integer type to avoid problems with some AV_WB32() implementations

2015-03-02 Thread Michael Niedermayer
On Sun, Mar 01, 2015 at 08:07:46PM -0500, Ronald S. Bultje wrote: Hi, On Sun, Mar 1, 2015 at 7:38 PM, Michael Niedermayer michae...@gmx.at wrote: alternatively lrint(f)() could be used or various other options Signed-off-by: Michael Niedermayer michae...@gmx.at ---

Re: [FFmpeg-devel] FFmpeg 2.6 release

2015-03-02 Thread Robert Krüger
On Sun, Mar 1, 2015 at 5:53 PM, Michael Niedermayer michae...@gmx.at wrote: Hi all its a while since FFmpeg 2.5, so its getting time to make 2.6 if you want something in it or something fixed, now is your last chance ;) About the name if noone suggests something then ill pick a random

Re: [FFmpeg-devel] FFmpeg 2.6 release

2015-03-02 Thread Michael Niedermayer
On Mon, Mar 02, 2015 at 05:41:41PM +0100, Robert Krüger wrote: On Sun, Mar 1, 2015 at 5:53 PM, Michael Niedermayer michae...@gmx.at wrote: Hi all its a while since FFmpeg 2.5, so its getting time to make 2.6 if you want something in it or something fixed, now is your last chance ;)

Re: [FFmpeg-devel] [PATCH] avcodec/rv10: check size of s-mb_width * s-mb_height

2015-03-02 Thread Michael Niedermayer
On Mon, Mar 02, 2015 at 05:52:52PM +0100, Andreas Cadhalpun wrote: Hi, encoding large frames with rv10 currently results in an assertion, because the number of macro blocks doesn't fit into 12 bits. Attached patch avoids this by just not trying to write the macro block number in that

Re: [FFmpeg-devel] [PATCH] mxfenc: Adjust product name when opatom muxer is used

2015-03-02 Thread Mark Reid
On Mon, Mar 2, 2015 at 1:46 AM, Robert Krüger krue...@lesspain.de wrote: On Sun, Mar 1, 2015 at 10:43 PM, Mark Reid mindm...@gmail.com wrote: On Sun, Mar 1, 2015 at 4:06 AM, Robert Krüger krue...@lesspain.de wrote: Currently the product name that ends up in mxf files muxed using the

[FFmpeg-devel] [PATCH] hevc/sao: use unaligned copy

2015-03-02 Thread Christophe Gisquet
Such calls as copy_CTB(dst - (left_pixels sh), ... may actually use completely unaligned addresses, causing issues on platforms such as ARM where 64bits copies have alignment requirements. In such cases, AC_COPY64U should be use instead. -- Christophe From

Re: [FFmpeg-devel] [PATCH] avcodec/rv10: check size of s-mb_width * s-mb_height

2015-03-02 Thread Andreas Cadhalpun
On 02.03.2015 18:35, Michael Niedermayer wrote: On Mon, Mar 02, 2015 at 05:52:52PM +0100, Andreas Cadhalpun wrote: Hi, encoding large frames with rv10 currently results in an assertion, because the number of macro blocks doesn't fit into 12 bits. Attached patch avoids this by just not trying

Re: [FFmpeg-devel] [PATCH] avcodec/rv10: check size of s-mb_width * s-mb_height

2015-03-02 Thread Michael Niedermayer
On Mon, Mar 02, 2015 at 07:18:10PM +0100, Andreas Cadhalpun wrote: On 02.03.2015 18:35, Michael Niedermayer wrote: On Mon, Mar 02, 2015 at 05:52:52PM +0100, Andreas Cadhalpun wrote: Hi, encoding large frames with rv10 currently results in an assertion, because the number of macro blocks

Re: [FFmpeg-devel] [PATCH] hevc/sao: use unaligned copy

2015-03-02 Thread Michael Niedermayer
On Mon, Mar 02, 2015 at 07:21:25PM +0100, Christophe Gisquet wrote: Such calls as copy_CTB(dst - (left_pixels sh), ... may actually use completely unaligned addresses, causing issues on platforms such as ARM where 64bits copies have alignment requirements. In such cases, AC_COPY64U should

Re: [FFmpeg-devel] [PATCH] avcodec/rv10: check size of s-mb_width * s-mb_height

2015-03-02 Thread Andreas Cadhalpun
On 02.03.2015 19:37, Michael Niedermayer wrote: On Mon, Mar 02, 2015 at 07:18:10PM +0100, Andreas Cadhalpun wrote: So would you prefer using avpriv_report_missing_feature, returning an error and checking the return value in libavcodec/mpegvideo_enc.c? if you dont have easy access to the real

[FFmpeg-devel] [PATCH] pthread: Fix ff_thread_get_format issues when called outside frame decode.

2015-03-02 Thread Reimar Döffinger
When ff_thread_get_format is called from the main thread, e.g. during codec init it will access the thread_ctx as a PerThreadContext even though it is a FrameThreadContext. Catch this case and add asserts to protect against similar issues in other places. In addition, when ff_thread_get_format is

[FFmpeg-devel] [PATCH] acvodec/lipopenjpeg: Improve XYZ color space detection

2015-03-02 Thread Vilius Grigaliūnas
Input files in XYZ color space are incorrecly detected as RGB which results in incorrect output colors. This changes pixel format detection to try XYZ before RGB when color space provided by libopenjepg is unknown. --- libavcodec/libopenjpegdec.c | 12 +++- 1 file changed, 7

[FFmpeg-devel] [PATCH] avcodec/webp: validate the distance prefix code

2015-03-02 Thread Andreas Cadhalpun
Hi, according to the WebP Lossless Bitstream Specification [1] the highest allowed value for the prefix code is 39. Attached patch adds a check for this to avoid crashes decoding broken files. Best regards, Andreas 1:

[FFmpeg-devel] [PATCH] avcodec/proresenc_kostya: fix coded_frame handling

2015-03-02 Thread Clément Bœsch
--- libavcodec/proresenc_kostya.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/libavcodec/proresenc_kostya.c b/libavcodec/proresenc_kostya.c index 59f73fb..440cc8b 100644 --- a/libavcodec/proresenc_kostya.c +++ b/libavcodec/proresenc_kostya.c @@ -914,7 +914,7