[FFmpeg-devel] [PATCH] zscale: Fix memory leak

2017-08-05 Thread Jkldjfset
Empty Message 0001-avfilter-zscale-fix-memory-leak.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH v2] avfilter/vf_ssim: fix temp size calculation

2017-08-05 Thread Muhammad Faiz
On Sat, Aug 5, 2017 at 2:48 PM, Paul B Mahol wrote: > On 8/4/17, Muhammad Faiz wrote: >> On Thu, Aug 3, 2017 at 1:53 PM, Tobias Rapp wrote: >>> On 03.08.2017 03:03, Muhammad Faiz wrote: Also use av_mallocz_array. Fix

[FFmpeg-devel] [PATCH] avcodec/dirac_dwt: Fixes integer overflows in COMPOSE_DAUB97*

2017-08-05 Thread Michael Niedermayer
Fix multiple: runtime error: signed integer overflow: 6497 * 3409630 cannot be represented in type 'int' Fixes: 2819/clusterfuzz-testcase-minimized-4743700301217792 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael

Re: [FFmpeg-devel] [PATCH] avutil/frame: allow only one element per type in frame side data

2017-08-05 Thread James Almer
On 7/29/2017 1:12 AM, James Almer wrote: > On 7/25/2017 9:10 PM, James Almer wrote: >> Same rationale as with packet side data, it was never meant to do otherwise >> as av_frame_get_side_data returns the first entry it finds of a given type. >> >> Based on code from libavformat's

[FFmpeg-devel] [PATCH] avformat/mov: prevent duplication of first fragment's ctts_data

2017-08-05 Thread Daniel Glöckner
MP4 files with fragments might have the first moof box that is mentioned in a fragment index before the first mdat box. Since it is then already parsed by mov_read_header, we have to make sure that mov_switch_root will not parse it again when seeking by setting the headers_read flag in the index.

Re: [FFmpeg-devel] libavcodec/exr : add SSE2 simd for reorder pixels (WIP)

2017-08-05 Thread Ivan Kalvachev
On 8/5/17, Martin Vignali wrote: > Hello, > > Based on pull request in the official openexr library : > https://github.com/openexr/openexr/pull/229/commits/4198128397c033d4f69e5cc0833195da500c31cf > > i try to port the reorder part (used in zip and rle decompression), to

Re: [FFmpeg-devel] [PATCH] doc/filters: document the unstability of the shorthand options notation.

2017-08-05 Thread James Almer
On 8/5/2017 6:17 PM, Nicolas George wrote: > L'octidi 18 thermidor, an CCXXV, James Almer a écrit : >> Indeed, but that's something that should have been done once AVOptions >> gained the shorthand feature with the CLI, not several years down the >> line and hundreds of scripts and tutorials

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/h264_slice: Fix overflow in slice offset

2017-08-05 Thread Michael Niedermayer
On Fri, Aug 04, 2017 at 04:05:24AM +0200, Michael Niedermayer wrote: > Fixes: runtime error: signed integer overflow: 1610612736 * 2 cannot be > represented in type 'int' > Fixes: 2817/clusterfuzz-testcase-minimized-5289691240726528 > > Found-by: continuous fuzzing process >

Re: [FFmpeg-devel] [PATCH] doc/filters: document the unstability of the shorthand options notation.

2017-08-05 Thread Nicolas George
L'octidi 18 thermidor, an CCXXV, James Almer a écrit : > Indeed, but that's something that should have been done once AVOptions > gained the shorthand feature with the CLI, not several years down the > line and hundreds of scripts and tutorials potentially considering it a > fixed syntax. > > At

Re: [FFmpeg-devel] [PATCH] avformat/utils: fix memory leak in avformat_free_context

2017-08-05 Thread Michael Niedermayer
On Thu, Aug 03, 2017 at 09:29:37AM -0700, Steven Siloti wrote: > On Mon, Jul 31, 2017 at 11:08 AM, Steven Siloti > wrote: > > > The pointer to the packet queue is stored in the internal structure > > so the queue needs to be flushed before internal is freed. > > > >

Re: [FFmpeg-devel] [PATCH v2] swscale: fix gbrap16 alpha channel issues

2017-08-05 Thread Michael Niedermayer
On Thu, Aug 03, 2017 at 04:21:54PM +0100, James Cowgill wrote: > Fixes filter-pixfmts-scale test failing on big-endian systems due to > alpSrc not being cast to (const int32_t**). > > Also fixes distortions in the output alpha channel values by copying the > alpha channel code from the rgba64

[FFmpeg-devel] libavcodec/exr : add SSE2 simd for reorder pixels (WIP)

2017-08-05 Thread Martin Vignali
Hello, Based on pull request in the official openexr library : https://github.com/openexr/openexr/pull/229/commits/4198128397c033d4f69e5cc0833195da500c31cf i try to port the reorder part (used in zip and rle decompression), to asm Tested on OS X pass fate test for me I test with this command :

Re: [FFmpeg-devel] [PATCH] Implement NewTek NDI support

2017-08-05 Thread Marton Balint
On Thu, 3 Aug 2017, Maksym Veremeyenko wrote: On 30.07.2017 17:19, Marton Balint wrote: [...] Subject: [PATCH] lavd: implement NewTek NDI muxer/demuxer support Hi, About the warnings for the bottom field order and the negative linesize, I still think it is better to reject them (return an

[FFmpeg-devel] [PATCH] Add macros used in opus_pvq_search to x86util.asm

2017-08-05 Thread Ivan Kalvachev
Improved version of VBROADCASTSS that works like the avx2 instruction. Emulation of vpbroadcastd. Horizontal sum HSUMPS that places the result in all elements. Emulation of blendvps and pblendvb. From cf4dc8fcd974a845b91aaa8685c06fa145b01786 Mon Sep 17 00:00:00 2001 From: Ivan Kalvachev

Re: [FFmpeg-devel] [PATCH] doc/filters: document the unstability of the shorthand options notation.

2017-08-05 Thread James Almer
On 8/5/2017 2:15 PM, Nicolas George wrote: > L'octidi 18 thermidor, an CCXXV, James Almer a écrit : >> Users may not know that it's a shortcut to begin with. > > It is exactly the reason I am proposing to document it here. Indeed, but that's something that should have been done once AVOptions

Re: [FFmpeg-devel] [PATCH] doc/filters: document the unstability of the shorthand options notation.

2017-08-05 Thread Nicolas George
L'octidi 18 thermidor, an CCXXV, James Almer a écrit : > Users may not know that it's a shortcut to begin with. It is exactly the reason I am proposing to document it here. Regards, -- Nicolas George signature.asc Description: Digital signature

Re: [FFmpeg-devel] [PATCH] doc/filters: document the unstability of the shorthand options notation.

2017-08-05 Thread James Almer
On 8/5/2017 1:42 PM, Nicolas George wrote: > L'octidi 18 thermidor, an CCXXV, Michael Niedermayer a écrit : >> I think you do not realize how annoying this would be in practice >> >> Users do not know all these nuances that one syntax is stable and work >> and one works but is not future proof. It

Re: [FFmpeg-devel] [PATCH] lavf/movenc.c: Set sgpd and sbgp atoms to represent decoder delay for AAC.

2017-08-05 Thread Sasi Inguva
To the best of my knowledge, all fate changes are benign. All of them correspond to a size increase of 54 bytes, which is the total size of the new atoms added. On Sat, Aug 5, 2017 at 9:27 AM, Derek Buitenhuis wrote: > On 8/4/2017 9:35 PM, Sasi Inguva wrote: > >

Re: [FFmpeg-devel] [PATCH] doc/filters: document the unstability of the shorthand options notation.

2017-08-05 Thread Nicolas George
L'octidi 18 thermidor, an CCXXV, Michael Niedermayer a écrit : > I think you do not realize how annoying this would be in practice > > Users do not know all these nuances that one syntax is stable and work > and one works but is not future proof. It also violates the priniple of > least surprise.

Re: [FFmpeg-devel] [PATCH] lavf/movenc.c: Set sgpd and sbgp atoms to represent decoder delay for AAC.

2017-08-05 Thread Derek Buitenhuis
On 8/5/2017 5:27 PM, Derek Buitenhuis wrote: > Probably OK, if FATE changes have been confirmed to be benign. Woops, replied to the wrong patch version. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] lavf/movenc.c: Set sgpd and sbgp atoms to represent decoder delay for AAC.

2017-08-05 Thread Derek Buitenhuis
On 8/4/2017 9:35 PM, Sasi Inguva wrote: > According to > https://developer.apple.com/library/content/documentation/QuickTime/QTFF/QTFFAppenG/QTFFAppenG.html > and ISO-IEC-14496-12 Section 10.1.1.1 and 10.1.1.3 > > Signed-off-by: Sasi Inguva > --- > libavformat/movenc.c

Re: [FFmpeg-devel] [PATCH] lavf/movenc.c: Set sgpd and sbgp atoms to represent decoder delay for AAC.

2017-08-05 Thread Derek Buitenhuis
On 8/4/2017 7:46 PM, Sasi Inguva wrote: > According to > https://developer.apple.com/library/content/documentation/QuickTime/QTFF/QTFFAppenG/QTFFAppenG.html > > Signed-off-by: Sasi Inguva > --- > libavformat/movenc.c| 70 >

Re: [FFmpeg-devel] [PATCH 2/8] build: treat crystalhd like other hwaccels

2017-08-05 Thread Philip Langdale
On Fri, 28 Jul 2017 11:59:13 + Philip Langdale wrote: > On 2017-07-28 11:51, Clément Bœsch wrote: > > From: Clément Bœsch > > > > --- > > configure | 7 --- > > 1 file changed, 4 insertions(+), 3 deletions(-) > > > > diff --git a/configure

Re: [FFmpeg-devel] [PATCH] doc/filters: document the unstability of the shorthand options notation.

2017-08-05 Thread Michael Niedermayer
On Sat, Aug 05, 2017 at 03:42:08PM +0200, Nicolas George wrote: > L'octidi 18 thermidor, an CCXXV, Michael Niedermayer a écrit : > > This is ambigous and if interpreted litterally > > then even a scale=320:240 would no longer be guranteed to work > > but would require scale=width=320:height=240 to

Re: [FFmpeg-devel] [PATCH]v6 Opus Pyramid Vector Quantization Search in x86 SIMD asm

2017-08-05 Thread Ivan Kalvachev
On 8/5/17, Hendrik Leppkes wrote: > On Sat, Aug 5, 2017 at 12:21 PM, Ivan Kalvachev > wrote: >> On 8/5/17, Rostislav Pehlivanov wrote: >>> On 4 August 2017 at 23:58, Ivan Kalvachev wrote: >>> >> I

Re: [FFmpeg-devel] [PATCH] doc/filters: document the unstability of the shorthand options notation.

2017-08-05 Thread Nicolas George
L'octidi 18 thermidor, an CCXXV, Michael Niedermayer a écrit : > This is ambigous and if interpreted litterally > then even a scale=320:240 would no longer be guranteed to work > but would require scale=width=320:height=240 to be used. That is intentional. Note that w=320:h=240 is enough: four

Re: [FFmpeg-devel] [PATCH] doc/filters: document the unstability of the shorthand options notation.

2017-08-05 Thread Michael Niedermayer
On Sat, Aug 05, 2017 at 11:42:06AM +0200, Nicolas George wrote: > Signed-off-by: Nicolas George > --- > doc/filters.texi | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/doc/filters.texi b/doc/filters.texi > index 7e5a9a625a..886cd5a7e8 100644 > ---

[FFmpeg-devel] [PATCH] add gray9 support

2017-08-05 Thread Paul B Mahol
Hi, patches attached. 0001-libavutil-add-GRAY9-pixel-format.patch Description: Binary data 0002-libswscale-add-gray9-support.patch Description: Binary data 0003-avfilter-vf_extractplanes-add-9-bitdepth-support.patch Description: Binary data ___

Re: [FFmpeg-devel] [PATCH] avfilter/vf_overlay: fix alpha blending for planar formats with a transparent background

2017-08-05 Thread Michael Niedermayer
On Fri, Aug 04, 2017 at 10:23:55AM +0200, Marton Balint wrote: > When the background had an alpha channel, the old code in blend_plane > calculated premultiplied alpha from the destination plane colors instead of > the > destination alpha. > > Also the calculation of the output alpha should only

Re: [FFmpeg-devel] [PATCH]v6 Opus Pyramid Vector Quantization Search in x86 SIMD asm

2017-08-05 Thread Hendrik Leppkes
On Sat, Aug 5, 2017 at 12:21 PM, Ivan Kalvachev wrote: > On 8/5/17, Rostislav Pehlivanov wrote: >> On 4 August 2017 at 23:58, Ivan Kalvachev wrote: >> >>> >>> >> I had it mixed before, but I changed it to be consistent. >>> >>

Re: [FFmpeg-devel] [PATCH] avfilter/vf_premultiply: add inplace mode

2017-08-05 Thread Paul B Mahol
On 8/5/17, Nicolas George wrote: > Le septidi 17 thermidor, an CCXXV, Paul B Mahol a écrit : >> From f952d9a2c87f7c2387d509e986aee5adb424d9d2 Mon Sep 17 00:00:00 2001 >> From: Paul B Mahol >> Date: Fri, 4 Aug 2017 22:28:53 +0200 >> Subject: [PATCH]

Re: [FFmpeg-devel] [PATCH]v6 Opus Pyramid Vector Quantization Search in x86 SIMD asm

2017-08-05 Thread Ivan Kalvachev
On 8/5/17, Rostislav Pehlivanov wrote: > On 4 August 2017 at 23:58, Ivan Kalvachev wrote: > >> >> >> I had it mixed before, but I changed it to be consistent. >> >> Some new avx instruction are not overloaded or >> >> available without their "v-"

[FFmpeg-devel] [PATCH v3] libaf/hlsenc: allow dynamic encryption key rotation

2017-08-05 Thread DeHackEd
Makes behaviour of 805ce25b1d2f optional, re-enables HLS key rotation feature Signed-off-by: DHE --- doc/muxers.texi | 7 ++- libavformat/hlsenc.c | 4 +++- 2 files changed, 9 insertions(+), 2 deletions(-) v1->v2: Actually works this time v2->v3: Documentation fix,

Re: [FFmpeg-devel] [PATCH] avfilter/vf_premultiply: add inplace mode

2017-08-05 Thread Nicolas George
Le septidi 17 thermidor, an CCXXV, Paul B Mahol a écrit : > From f952d9a2c87f7c2387d509e986aee5adb424d9d2 Mon Sep 17 00:00:00 2001 > From: Paul B Mahol > Date: Fri, 4 Aug 2017 22:28:53 +0200 > Subject: [PATCH] avfilter/vf_premultiply: add inplace mode > > --- >

[FFmpeg-devel] [PATCH] doc/filters: document the unstability of the shorthand options notation.

2017-08-05 Thread Nicolas George
Signed-off-by: Nicolas George --- doc/filters.texi | 5 + 1 file changed, 5 insertions(+) diff --git a/doc/filters.texi b/doc/filters.texi index 7e5a9a625a..886cd5a7e8 100644 --- a/doc/filters.texi +++ b/doc/filters.texi @@ -161,6 +161,11 @@ follow the same constraints

Re: [FFmpeg-devel] [PATCH 07/14] lavfi/vf_overlay: use framesync2 options.

2017-08-05 Thread Nicolas George
L'octidi 18 thermidor, an CCXXV, Michael Niedermayer a écrit : > The documentation says > @section overlay > [...] > @table @option > @item x > @item y > [...] > @item eof_action > > Thats the order used in the example Yes. But it does not work like that for all the options. So basically, you

Re: [FFmpeg-devel] [PATCH 07/14] lavfi/vf_overlay: use framesync2 options.

2017-08-05 Thread Nicolas George
Le septidi 17 thermidor, an CCXXV, Paul B Mahol a écrit : > Also what about to porting one of simple filter(just filter_frame > call) to activate API, > i can not do it myself unless someone show me how. Now that the new design has proved to work, rewriting doc/filter_design.txt is near the top

Re: [FFmpeg-devel] [PATCH v2] avfilter/vf_ssim: fix temp size calculation

2017-08-05 Thread Paul B Mahol
On 8/4/17, Muhammad Faiz wrote: > On Thu, Aug 3, 2017 at 1:53 PM, Tobias Rapp wrote: >> On 03.08.2017 03:03, Muhammad Faiz wrote: >>> >>> Also use av_mallocz_array. >>> Fix Ticket6519. >>> >>> Signed-off-by: Muhammad Faiz >>> --- >>>