[FFmpeg-devel] [PATCH] avcodec/avpacket: Check metadata key in av_packet_unpack_dictionary()

2017-02-26 Thread Michael Niedermayer
Fixes timeout Fixes: 501/clusterfuzz-testcase-5672752870588416 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/avpacket.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/liba

[FFmpeg-devel] [PATCH] vf_hwupload: Add missing return value check

2017-02-26 Thread Jun Zhao
From e60a958c78ba084e0745acebec5125394387c5b1 Mon Sep 17 00:00:00 2001 From: Jun Zhao Date: Mon, 27 Feb 2017 10:08:36 +0800 Subject: [PATCH] vf_hwupload: Add missing return value checks Add missing return value checks to fix the build warning. Signed-off-by: Jun Zhao --- libavfilter/vf_hwuploa

[FFmpeg-devel] [PATCH] vf_hwupload: Add missing return value checks

2017-02-26 Thread Jun Zhao
From e60a958c78ba084e0745acebec5125394387c5b1 Mon Sep 17 00:00:00 2001 From: Jun Zhao Date: Mon, 27 Feb 2017 10:08:36 +0800 Subject: [PATCH] vf_hwupload: Add missing return value checks Add missing return value checks to fix the build warning. Signed-off-by: Jun Zhao --- libavfilter/vf_hwuploa

Re: [FFmpeg-devel] [PATCH] add locale month names to av_small_strptime

2017-02-26 Thread Micah Galizia
On Sun, Feb 26, 2017 at 11:12 AM, Michael Niedermayer wrote: > On Thu, Feb 23, 2017 at 09:34:28PM -0500, Micah Galizia wrote: >> Hello, >> >> Is someone able to take a look at this and accept or reject it -- its >> been a few days since I submitted. > > applied > > thx Awesome -- thank you. On to

Re: [FFmpeg-devel] [PATCH 3/4] avcodec/h264idct_template: Fix multiple runtime error: signed integer overflow

2017-02-26 Thread Steven Liu
2017-02-27 3:28 GMT+08:00 Michael Niedermayer : > Fixes: 677/clusterfuzz-testcase-6635120628858880 > > Found-by: continuous fuzzing process https://github.com/google/oss- > fuzz/tree/master/targets/ffmpeg > Signed-off-by: Michael Niedermayer > --- > libavcodec/h264idct_template.c | 26 ++

Re: [FFmpeg-devel] [PATCH 2/4] avdevice/decklink_enc: add support to specify field order

2017-02-26 Thread Marton Balint
On Sat, 25 Feb 2017, Marton Balint wrote: [...] Ok, will push in 1-2 days. Series applied. Regards, Marton ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 4/5] avcodec/eac3dec: Fix runtime error: left shift of negative value -3

2017-02-26 Thread Kieran Kunhya
On Sat, 25 Feb 2017 at 20:08 Michael Niedermayer wrote: > Fixes: 672/clusterfuzz-testcase-5595018867769344 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg > Signed-off-by >

[FFmpeg-devel] [PATCH 4/4] avcodec/dca: Fix multiple runtime error: signed integer overflow

2017-02-26 Thread Michael Niedermayer
Fixes: 680/clusterfuzz-testcase-5416627266912256 Fixes: 681/clusterfuzz-testcase-5013323462475776 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/dca_xll.c | 4 ++-- libavcodec/dcadsp.c | 2 +-

[FFmpeg-devel] [PATCH 3/4] avcodec/h264idct_template: Fix multiple runtime error: signed integer overflow

2017-02-26 Thread Michael Niedermayer
Fixes: 677/clusterfuzz-testcase-6635120628858880 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/h264idct_template.c | 26 +- 1 file changed, 13 insertions(+), 13 deletio

[FFmpeg-devel] [PATCH 1/4] avcodec/mpeg4videodec: Fix runtime error: left shift of negative value -2650

2017-02-26 Thread Michael Niedermayer
Fixes: 674/clusterfuzz-testcase-6713275880308736 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/mpeg4videodec.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcode

[FFmpeg-devel] [PATCH 2/4] avcodec/pictordec: Check plane value before doing value/mask computations

2017-02-26 Thread Michael Niedermayer
Fixes integer overflow Fixes: 675/clusterfuzz-testcase-6722971232108544 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/pictordec.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) dif

Re: [FFmpeg-devel] [PATCH] add locale month names to av_small_strptime

2017-02-26 Thread Michael Niedermayer
On Thu, Feb 23, 2017 at 09:34:28PM -0500, Micah Galizia wrote: > Hello, > > Is someone able to take a look at this and accept or reject it -- its > been a few days since I submitted. applied thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB When the tyrant h

Re: [FFmpeg-devel] [PATCH 4/5] avcodec/eac3dec: Fix runtime error: left shift of negative value -3

2017-02-26 Thread Michael Niedermayer
On Sat, Feb 25, 2017 at 10:02:21PM +0100, Paul B Mahol wrote: > On 2/25/17, Michael Niedermayer wrote: > > Fixes: 672/clusterfuzz-testcase-5595018867769344 > > > > Found-by: continuous fuzzing process > > https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg > > Signed-off-by: Michael Nied

Re: [FFmpeg-devel] [PATCH 2/5] avcodec/ituh263dec: Check for the bitstream end in ff_h263_decode_mb()

2017-02-26 Thread Michael Niedermayer
On Sun, Feb 26, 2017 at 10:06:18AM +0100, Paul B Mahol wrote: > On 2/25/17, Michael Niedermayer wrote: > > On Sat, Feb 25, 2017 at 10:03:58PM +0100, Paul B Mahol wrote: > >> On 2/25/17, Michael Niedermayer wrote: > >> > Fixes invalid shift > >> > > >> > Fixes: 670/clusterfuzz-testcase-48520210667

Re: [FFmpeg-devel] [PATCH 3/5] avcodec/mpeg12dec: Fix runtime error: left shift of negative value -2

2017-02-26 Thread Michael Niedermayer
On Sat, Feb 25, 2017 at 10:01:17PM +0100, Paul B Mahol wrote: > On 2/25/17, Michael Niedermayer wrote: > > 671/clusterfuzz-testcase-4990381827555328 > > > > Found-by: continuous fuzzing process > > https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg > > Signed-off-by: Michael Niedermayer

Re: [FFmpeg-devel] [PATCH 5/5] avcodec/flacdsp: Fix: runtime error: signed integer overflow: -1027555328 + -1226681270 cannot be represented in type 'int'

2017-02-26 Thread Michael Niedermayer
On Sun, Feb 26, 2017 at 10:06:57AM +0100, Paul B Mahol wrote: > On 2/25/17, Michael Niedermayer wrote: > > On Sat, Feb 25, 2017 at 10:00:36PM +0100, Paul B Mahol wrote: > >> On 2/25/17, Michael Niedermayer wrote: > >> > Fixes: 673/clusterfuzz-testcase-5948736536576000 > >> > > >> > Found-by: cont

Re: [FFmpeg-devel] [PATCH v2 1/3] ffmpeg_opt: pass output framerate as a hint to the encoder

2017-02-26 Thread Michael Niedermayer
On Mon, Feb 20, 2017 at 04:05:00PM +0100, Tobias Rapp wrote: > On 20.02.2017 15:09, Mark Thompson wrote: > >On 06/02/17 12:33, Tobias Rapp wrote: > >>Sets framerate field in output codec context if explicitly specified on > >>the command-line. > >> > >>Signed-off-by: Tobias Rapp > >>--- > >> ffmpe

[FFmpeg-devel] [PATCH]lavc/internal: Constify AVPacket* in AVCodecInternal.

2017-02-26 Thread Carl Eugen Hoyos
Hi! Attached patch fixes another warning when compiling libavcodec/utils.o. Please comment, Carl Eugen From 1a77bc829eb2a906e175f296821ee7f91f19724a Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Sun, 26 Feb 2017 12:34:41 +0100 Subject: [PATCH] lavc/internal: Constify AVPacket* in AVCodec

[FFmpeg-devel] [PATCH 3/2] Re: deduplicated [PATCH] Cinepak: speed up decoding several-fold, depending on the scenario, by supporting multiple output pixel formats.

2017-02-26 Thread u-9iep
This extra patch "beyond the series" is being posted for the benefit of a casual reader who needs extremely fast/lightweight video decoding of prearranged videos, with existing or/and mainstream applications (e.g. like mplayer). This patch is vital to make the cinepak decoding speedup (the matter

Re: [FFmpeg-devel] [PATCH]lswr/rematrix: Remove an aggressive loop optimization / undefined behaviour

2017-02-26 Thread Carl Eugen Hoyos
2017-02-24 22:24 GMT+01:00 Michael Niedermayer : > On Fri, Feb 24, 2017 at 11:21:54AM +0100, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch fixes a gcc warning (tested 6.3.0). >> >> Please comment, Carl Eugen > >> rematrix.c |7 --- >> 1 file changed, 4 insertions(+), 3 deletions(-) >

Re: [FFmpeg-devel] [PATCH]lavc/avpacket: Make pkt parameter of av_packet_get_side_data() const

2017-02-26 Thread Carl Eugen Hoyos
2017-02-26 11:51 GMT+01:00 Nicolas George : > L'octidi 8 ventôse, an CCXXV, Carl Eugen Hoyos a écrit : >> Hi! >> >> I believe adding const to the declaration of av_packet_get_side_data() >> better reflects its intention, also fixes a warning. >> >> Please comment, Carl Eugen > >> From a17c4bab8cd21

Re: [FFmpeg-devel] [PATCH]lavc/avpacket: Make pkt parameter of av_packet_get_side_data() const

2017-02-26 Thread Nicolas George
L'octidi 8 ventôse, an CCXXV, Carl Eugen Hoyos a écrit : > Hi! > > I believe adding const to the declaration of av_packet_get_side_data() > better reflects its intention, also fixes a warning. > > Please comment, Carl Eugen > From a17c4bab8cd21c15e91f5efd03d5900eda29090b Mon Sep 17 00:00:00 200

[FFmpeg-devel] [PATCH]lavc/avpacket: Make pkt parameter of av_packet_get_side_data() const

2017-02-26 Thread Carl Eugen Hoyos
Hi! I believe adding const to the declaration of av_packet_get_side_data() better reflects its intention, also fixes a warning. Please comment, Carl Eugen From a17c4bab8cd21c15e91f5efd03d5900eda29090b Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Sun, 26 Feb 2017 11:39:07 +0100 Subject:

[FFmpeg-devel] [PATCH]lsws/slice: Move a misplaced const, fix a warning

2017-02-26 Thread Carl Eugen Hoyos
Hi! Attached patch fixes two warnings when compiling libswscale/slice.o. Please comment, Carl Eugen From d4d0a2c9c955d7dfd5bf9a91d31fcc3a1696e79d Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Sun, 26 Feb 2017 11:18:25 +0100 Subject: [PATCH] lsws/slice: Move a misplaced const. MIME-Versio

Re: [FFmpeg-devel] [PATCH 1/2] Re: deduplicated [PATCH] Cinepak: speed up decoding several-fold, depending on the scenario, by supporting multiple output pixel formats.

2017-02-26 Thread u-9iep
On Sat, Feb 25, 2017 at 09:27:05PM +0100, Paul B Mahol wrote: > On 2/25/17, u-9...@aetey.se wrote: > > Hello, > > > > Here comes the latest version of the patch, with adjustments > > made according to all substantial feedback comments. > > > > Among others the code has been further deduplicated to

[FFmpeg-devel] [PATCH]lavc/xface: Reorder conditions to silence a gcc warning

2017-02-26 Thread Carl Eugen Hoyos
Hi! I believe attached patch does not change the logic of the conditions but silences a (9 times shown, long-time) gcc warning. Please review, Carl Eugen From 2a634f0b1f265f96111895de8603f9deda91d3df Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Sun, 26 Feb 2017 11:03:50 +0100 Subject:

Re: [FFmpeg-devel] [PATCH 5/5] avcodec/flacdsp: Fix: runtime error: signed integer overflow: -1027555328 + -1226681270 cannot be represented in type 'int'

2017-02-26 Thread Paul B Mahol
On 2/25/17, Michael Niedermayer wrote: > On Sat, Feb 25, 2017 at 10:00:36PM +0100, Paul B Mahol wrote: >> On 2/25/17, Michael Niedermayer wrote: >> > Fixes: 673/clusterfuzz-testcase-5948736536576000 >> > >> > Found-by: continuous fuzzing process >> > https://github.com/google/oss-fuzz/tree/master

Re: [FFmpeg-devel] [PATCH 2/5] avcodec/ituh263dec: Check for the bitstream end in ff_h263_decode_mb()

2017-02-26 Thread Paul B Mahol
On 2/25/17, Michael Niedermayer wrote: > On Sat, Feb 25, 2017 at 10:03:58PM +0100, Paul B Mahol wrote: >> On 2/25/17, Michael Niedermayer wrote: >> > Fixes invalid shift >> > >> > Fixes: 670/clusterfuzz-testcase-4852021066727424 >> > >> > Found-by: continuous fuzzing process >> > https://github.c