Fixes: avcodec/aacps.c:511:40: runtime error: signed integer overflow:
1509077651 + 758068176 cannot be represented in type 'int'
Fixes: 2678/clusterfuzz-testcase-minimized-4702787684270080
Found-by: continuous fuzzing process
https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed
Fixes: runtime error: shift exponent 32 is too large for 32-bit type 'unsigned
int'
Fixes: 2698/clusterfuzz-testcase-minimized-4713541443518464
Found-by: continuous fuzzing process
https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer
---
libavcodec/
old:
165188 decicycles in rdft, 65536 runs, 0 skips
165865 decicycles in irdft, 65536 runs, 0 skips
new:
142487 decicycles in rdft, 65536 runs, 0 skips
141498 decicycles in irdft, 65536 runs, 0 skips
Signed-off-by: Muhammad Faiz
---
libavcodec/rdft.c | 14 ++
On Fri, Jul 21, 2017 at 04:39:37PM +0200, Tobias Rapp wrote:
> On 20.07.2017 22:21, Michael Niedermayer wrote:
> >On Wed, Jul 19, 2017 at 05:23:07PM +0200, Tobias Rapp wrote:
> >[...]
> >> fate/filter-video.mak | 16 ++
> >> ref/fate/filter-refcmp-psnr-rgb | 45
> >> ++
The environment variable used by frei0r filter is named FREI0R_PATH
not FREIOR_PATH.
0001-doc-filters-typo-in-frei0r.patch
Description: Binary data
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
On 7/21/17, Reimar Doeffinger wrote:
> On 21.07.2017, at 08:48, Paul B Mahol wrote:
>
>> On 7/20/17, Carl Eugen Hoyos wrote:
>>> 2017-07-20 17:46 GMT+02:00 Vittorio Giovara :
>>>
+[AV_PIX_FMT_YUV444F32LE] = { 0, 0 },
+[AV_PIX_FMT_YUV444F32BE] = { 0, 0 },
+[AV_PIX_FMT
On 21.07.2017, at 08:48, Paul B Mahol wrote:
> On 7/20/17, Carl Eugen Hoyos wrote:
>> 2017-07-20 17:46 GMT+02:00 Vittorio Giovara :
>>
>>> +[AV_PIX_FMT_YUV444F32LE] = { 0, 0 },
>>> +[AV_PIX_FMT_YUV444F32BE] = { 0, 0 },
>>> +[AV_PIX_FMT_YUVA444F32LE] = { 0, 0 },
>>> +[AV_PIX_FM
On 21.07.2017, at 16:26, Paras Chadha wrote:
> On Fri, Jul 21, 2017 at 3:27 AM, Carl Eugen Hoyos
> wrote:
>
>> 2017-07-20 21:46 GMT+02:00 Paras Chadha :
>>
>>> +case AV_PIX_FMT_RGB24:
>>> +case AV_PIX_FMT_RGBA:
>>> +for (k = 0; k < naxis3; k++) {
>>> +
On Fri, Jul 21, 2017 at 01:56:33PM +, Manojkumar Bhosale wrote:
> LGTM
applied
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
If you fake or manipulate statistics in a paper in physics you will never
get a job again.
If you fake or manipulate statistic
ticket-id: #6541
when use hls fmp4 muxer, the extention name is not .m4s, this
code can fix it.
Found-by: JohnPi
Signed-off-by: Steven Liu
---
libavformat/hlsenc.c | 47 +--
1 file changed, 41 insertions(+), 6 deletions(-)
diff --git a/libavformat/hls
On 7/21/2017 9:22 AM, foo86 wrote:
> ---
> MAINTAINERS | 4
> 1 file changed, 4 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 60aae12faa..ae0e08d121 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -159,8 +159,10 @@ Codecs:
>crystalhd.c Phi
On 7/21/2017 8:55 AM, foo86 wrote:
> On Fri, Jul 21, 2017 at 01:02:41AM -0300, James Almer wrote:
>> Will apply soon.
>
> LGTM.
Pushed, thanks!
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
... any chance for this patch to get applied? Quick link to original thread:
https://ffmpeg.org/pipermail/ffmpeg-devel/2017-June/211866.html
Rebased version of patch against latest master attached.
Kind regards,
Matthias
diff --git a/Changelog b/Changelog
index 187ae79..4fa0870 100644
--- a/Chan
Le tridi 3 thermidor, an CCXXV, Paras Chadha a écrit :
> The only difference between first image and rest is that, the first image
> will always contain the keyword, SIMPLE = T as first keyword while all
> other images will have XTENSION = 'IMAGE ' as first keyword.
Then I would say that "SIMPLE =
On 20.07.2017 22:21, Michael Niedermayer wrote:
On Wed, Jul 19, 2017 at 05:23:07PM +0200, Tobias Rapp wrote:
[...]
fate/filter-video.mak | 16 ++
ref/fate/filter-refcmp-psnr-rgb | 45
ref/fate/filter-refcmp-psnr-yuv | 45 +
Le tridi 3 thermidor, an CCXXV, Paras Chadha a écrit :
> > > AVOutputFormat ff_image2pipe_muxer = {
> > > .name = "image2pipe",
> > > .long_name = NULL_IF_CONFIG_SMALL("piped image2 sequence"),
> > > +.extensions = "fits",
> > This is probably wrong. Did you intend
On Fri, Jul 21, 2017 at 3:27 AM, Carl Eugen Hoyos
wrote:
> 2017-07-20 21:46 GMT+02:00 Paras Chadha :
>
> > +case AV_PIX_FMT_RGB24:
> > +case AV_PIX_FMT_RGBA:
> > +for (k = 0; k < naxis3; k++) {
> > +for (i = 0; i < avctx->height; i++) {
On Fri, Jul 21, 2017 at 1:45 AM, James Almer wrote:
> On 7/20/2017 4:46 PM, Paras Chadha wrote:
> > diff --git a/libavformat/img2enc.c b/libavformat/img2enc.c
> > index 1297b1a..25283cc 100644
> > --- a/libavformat/img2enc.c
> > +++ b/libavformat/img2enc.c
> > @@ -237,6 +237,7 @@ AVOutputFormat f
Le duodi 2 thermidor, an CCXXV, Paras Chadha a écrit :
> +int64_t t, size = 1;
> +if (!strncmp(keyword, "BLANK", 5) && sscanf(value, "%ld", &t) == 1) {
This %ld is not the correct format specifier for int64_t. It would be
something like %"SCNd64".
Regards,
--
Nicolas George
sig
On Wed, Jul 19, 2017 at 02:44:29AM +0200, Michael Niedermayer wrote:
> Fixes OOM
> Fixes: 2674/clusterfuzz-testcase-minimized-4999700518273024
>
> Found-by: continuous fuzzing process
> https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
> Signed-off-by: Michael Niedermayer
> ---
> l
On Mon, Jul 17, 2017 at 09:17:09AM -0600, Tyler Jones wrote:
> The encoder will attempt to determine the existence of transient
> signals by applying a 4th order highpass filter to remove dominant
> low frequency waveforms. Frames are then split up into blocks
> where the variance is calculated and
On Fri, Jul 21, 2017 at 04:09:29PM +0200, Clément Bœsch wrote:
> On Fri, Jul 21, 2017 at 03:03:19PM +0100, Derek Buitenhuis wrote:
> > On 7/21/2017 2:55 PM, Clément Bœsch wrote:
> > > It's basically the same layout for every stsd sample description. Check
> > > the other tags that extend that layou
On Sat, Jul 15, 2017 at 07:57:27PM +0200, Michael Niedermayer wrote:
> Requested-by: Kieran Kunhya
>
> Signed-off-by: Michael Niedermayer
> ---
> doc/undefined.txt | 47 +++
> 1 file changed, 47 insertions(+)
> create mode 100644 doc/undefined.txt
t
On Fri, Jul 21, 2017 at 03:03:19PM +0100, Derek Buitenhuis wrote:
> On 7/21/2017 2:55 PM, Clément Bœsch wrote:
> > It's basically the same layout for every stsd sample description. Check
> > the other tags that extend that layout, the also have that data ref idx to
> > 1. I think I copied from the
On 7/21/2017 2:55 PM, Clément Bœsch wrote:
> It's basically the same layout for every stsd sample description. Check
> the other tags that extend that layout, the also have that data ref idx to
> 1. I think I copied from the rtp code, but it's the same for the other as
> well. This gpmd atom just e
LGTM
From: ffmpeg-devel [ffmpeg-devel-boun...@ffmpeg.org] on behalf of
kaustubh.ra...@imgtec.com [kaustubh.ra...@imgtec.com]
Sent: Friday, July 21, 2017 11:54 AM
To: ffmpeg-devel@ffmpeg.org
Cc: Kaustubh Raste
Subject: [FFmpeg-devel] [PATCH] libavutil/mips:
On Fri, Jul 21, 2017 at 02:21:23PM +0100, Derek Buitenhuis wrote:
> On 7/21/2017 9:30 AM, Clément Bœsch wrote:
> > +avio_wb16(pb, 1); /* Data-reference index */
>
> Why is this whole atom hardcoded (i.e. with is '1').
>
It's basically the same layout for every stsd sample description. Check
Le tridi 3 thermidor, an CCXXV, Carl Eugen Hoyos a écrit :
> alpha_value = x * 255 / width;
> or
> alpha_value = y * 255 / height;
>
> Sorry for being unclear, Carl Eugen
Oh, you mean a gradient on the alpha channel. It is not really possible
without making the code more complex and I think signi
2017-07-21 15:43 GMT+02:00 Carl Eugen Hoyos :
> 2017-07-21 15:40 GMT+02:00 Nicolas George :
>> Le tridi 3 thermidor, an CCXXV, Carl Eugen Hoyos a écrit :
>>> The default you suggest is of course much better than the original
>>> default but ideally, some kind of "banding" should be used to allow
>>
Le tridi 3 thermidor, an CCXXV, Carl Eugen Hoyos a écrit :
> I meant value = width or value = height (but I am not saying this
> is necessarily a wise suggestion).
I am sorry, but now I really do not understand at all.
Regards,
--
Nicolas George
signature.asc
Description: Digital signature
2017-07-21 15:40 GMT+02:00 Nicolas George :
> Le tridi 3 thermidor, an CCXXV, Carl Eugen Hoyos a écrit :
>> The default you suggest is of course much better than the original
>> default but ideally, some kind of "banding" should be used to allow
>> testing transparency in libswscale, encoders and d
Le tridi 3 thermidor, an CCXXV, Carl Eugen Hoyos a écrit :
> The default you suggest is of course much better than the original
> default but ideally, some kind of "banding" should be used to allow
> testing transparency in libswscale, encoders and decoders.
Can you be a little more specific? I am
On 18 July 2017 at 02:12, Gerion Entrup wrote:
> Am Dienstag, 18. Juli 2017, 01:52:53 CEST schrieb Reimar Döffinger:
>> On 18.07.2017, at 00:59, James Almer wrote:
>>
>> > On 7/17/2017 7:49 PM, Moritz Barsnick wrote:
>> >> On Mon, Jul 10, 2017 at 13:53:02 +0300, Boris Pek wrote:
>> >>> Latest new
Le tridi 3 thermidor, an CCXXV, Derek Buitenhuis a écrit :
> +1 on exposing it as side data; it's consistent with previous APIs and doesn't
I will not oppose further; I do not buy the "Yet Another Field" argument
but I will debunk it another time.
> FWIW, literally every other library exposes ICC
2017-07-21 15:22 GMT+02:00 Michael Niedermayer :
> On Thu, Jul 20, 2017 at 09:54:05PM +0100, Mark Thompson wrote:
> PS: heres the patch without whitespace changes (much more readable)
Please use this variant and push a cosmetic change.
> commit a0f2f597f0922a37486732a961c94af0507af234
> Author:
On Fri, Jul 21, 2017 at 11:35:14AM +0800, efren yang wrote:
> It is applied to every glyph. There is no space between all words in some
> languages, for example Chinese, Japanese and so on. So a sentence will look
> very compactly. I think if we could set glyph spacing, it will adapt to more
> d
On 7/21/2017 12:55 AM, Rostislav Pehlivanov wrote:
>> All this could apply to a dedicated field. Using side data only brings a
>> type pruning of the actual type. Except:
>>
>>
> Yes, it could. However I still think having it as a side data is better
> since its the easiest way and that's what all
On Thu, Jul 20, 2017 at 09:54:05PM +0100, Mark Thompson wrote:
> ---
> On 20/07/17 01:33, Michael Niedermayer wrote:
> > Hi
> >
> > On Tue, Jul 18, 2017 at 11:01:01PM +, Mark Thompson wrote:
> >> ffmpeg | branch: master | Mark Thompson | Thu Jul 6
> >> 22:50:35 2017 +0100| [8a442d7a8a687a46
On 7/21/2017 9:30 AM, Clément Bœsch wrote:
> +avio_wb16(pb, 1); /* Data-reference index */
Why is this whole atom hardcoded (i.e. with is '1').
- Derek
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-d
On Fri, Jul 21, 2017 at 10:30:19AM +0200, Clément Bœsch wrote:
> From: Clément Bœsch
>
> See https://github.com/gopro/gpmf-parser for more information on the
> data stream itself.
> ---
> libavformat/movenc.c | 25 +
> 1 file changed, 25 insertions(+)
can you add a fate
On Thu, Jul 20, 2017 at 09:46:22PM +0100, Rostislav Pehlivanov wrote:
> Signed-off-by: Rostislav Pehlivanov
> ---
> libavcodec/pngdec.c | 45 +
> 1 file changed, 45 insertions(+)
>
> diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c
> index 083f61
On 07/21/2017 02:41 PM, Clément Bœsch wrote:
On Fri, Jul 21, 2017 at 02:26:03PM +0200, Jorge Ramirez-Ortiz wrote:
ffplay always requires user intervention via the GUI to close the
video at the end of the file.
This commit, stops playing and terminates the program when EOF is
received.
-autoexi
On Fri, Jul 21, 2017 at 5:56 PM, Jorge Ramirez-Ortiz <
jorge.ramirez-or...@linaro.org> wrote:
> ffplay always requires user intervention via the GUI to close the
> video at the end of the file.
>
You wish to make -autoexit the default?
___
ffmpeg-devel
On Fri, Jul 21, 2017 at 02:26:03PM +0200, Jorge Ramirez-Ortiz wrote:
> ffplay always requires user intervention via the GUI to close the
> video at the end of the file.
>
> This commit, stops playing and terminates the program when EOF is
> received.
-autoexit?
--
Clément B.
signature.asc
Des
On Wed, Jul 19, 2017 at 01:48:47PM +0200, Moritz Barsnick wrote:
> On Sat, Jul 01, 2017 at 17:03:46 +0300, foo86 wrote:
> > +static av_cold void init_tables(void)
> > +{
> > +int i, j;
> > +
> > +for (i = 1; i < 17; i++)
> > +mantissa_tab1[i][0] = 1.0 / (1 << i - 1);
> > +
> > +
ffplay always requires user intervention via the GUI to close the
video at the end of the file.
This commit, stops playing and terminates the program when EOF is
received.
---
ffplay.c | 11 ---
1 file changed, 8 insertions(+), 3 deletions(-)
diff --git a/ffplay.c b/ffplay.c
index c0b326
---
MAINTAINERS | 4
1 file changed, 4 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 60aae12faa..ae0e08d121 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -159,8 +159,10 @@ Codecs:
crystalhd.c Philip Langdale
cscd.cR
On Fri, Jul 21, 2017 at 01:02:41AM -0300, James Almer wrote:
> Will apply soon.
LGTM.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
On 20.07.2017 17:45, Vittorio Giovara wrote:
---
Updated according to Tobias' review.
Vittorio
doc/ffprobe.xsd| 5 +
ffprobe.c | 25 +
tests/ref/fate/ffprobe_compact | 16
tests/ref/fate/ffprobe_csv | 16
2017-07-21 11:50 GMT+02:00 Nicolas George :
> Note: until now, the "default" was 0, not 255. But it produces an ugly
> image, so I considered nobody really uses it like that and decided it was ok
> to change it to a saner default, but it can be discussed. It does not break
> our existing FATE test
On Thu, Jul 20, 2017 at 04:42:32PM +0800, efren_yang wrote:
> Hi, devel
>
> Thanks for reply. As you see, it is applied to every glyph. There is
> no space between all words in some languages, for example Chinese, Japanese
> and so on. So a sentence will look very compactly. I think if we
On Fri, Jul 21, 2017 at 12:28:13PM +0200, Nicolas George wrote:
> Le tridi 3 thermidor, an CCXXV, Michael Niedermayer a écrit :
> > this patch breaks the selftests on mips or maybe more precissely
> > it adds a test which fails
>
> Yes, I understand that. What I ask is who is responsible for the
>
On Fri, Jul 21, 2017 at 12:18:53PM +0200, Tobias Rapp wrote:
> On 21.07.2017 02:47, Michael Niedermayer wrote:
> >On Fri, Jul 21, 2017 at 01:04:03AM +0200, Nicolas George wrote:
> >>Le duodi 2 thermidor, an CCXXV, Michael Niedermayer a écrit :
> >>>breaks on mips (probably big endian issue)
> >>
>
From: Ashish Singh
Hi, libvmaf now has a pkg-config file. This patch makes libvmaf filter link to
pkg-config of libvmaf.
Signed-off-by: Ashish Singh
---
configure | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/configure b/configure
index 57694f3..aa8a785 100755
--- a/con
ticket-id: #6541
when use hls fmp4 muxer, the extention name is not .m4s, this
code can fix it.
Found-by: JohnPi
Signed-off-by: Steven Liu
---
libavformat/hlsenc.c | 44 ++--
1 file changed, 38 insertions(+), 6 deletions(-)
diff --git a/libavformat/hlsenc
Le tridi 3 thermidor, an CCXXV, Michael Niedermayer a écrit :
> this patch breaks the selftests on mips or maybe more precissely
> it adds a test which fails
Yes, I understand that. What I ask is who is responsible for the
failure: the test or the code being tested?
If somebody adds a test on a c
On 21.07.2017 02:47, Michael Niedermayer wrote:
On Fri, Jul 21, 2017 at 01:04:03AM +0200, Nicolas George wrote:
Le duodi 2 thermidor, an CCXXV, Michael Niedermayer a écrit :
breaks on mips (probably big endian issue)
Breaks on MIPS, or detects that something is already broken in MIPS and
nobo
Signed-off-by: Nicolas George
---
doc/filters.texi | 5 +
libavfilter/vsrc_testsrc.c | 12 +---
2 files changed, 14 insertions(+), 3 deletions(-)
Fixed the undefined shift, thanks Clément for noticing.
Added documentation.
Note: until now, the "default" was 0, not 255. B
From: Vodyannikov Aleksandr
Fixes: Ticket6546
Regression since: 54aaadf648073149f1ac34f56cbde4e6c5aa22ef
Signed-off-by: Michael Niedermayer
---
libavcodec/cfhd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/cfhd.c b/libavcodec/cfhd.c
index f78bad9ae1..5ea8f24
On Thu, Jul 20, 2017 at 05:42:51PM +0200, Nicolas George wrote:
[...]
> diff --git a/libavfilter/vsrc_testsrc.c b/libavfilter/vsrc_testsrc.c
> index c4a5ae3742..f94c3ed9dd 100644
> --- a/libavfilter/vsrc_testsrc.c
> +++ b/libavfilter/vsrc_testsrc.c
> @@ -66,6 +66,9 @@ typedef struct TestSourceConte
On 7/21/17, Nicolas George wrote:
> Le tridi 3 thermidor, an CCXXV, Paul B Mahol a ecrit :
>> We will just ignore you as usual.
>
> Will you please stop the ad-hominem attacks?
No I will not, until you and others stop being disrespectful.
___
ffmpeg-dev
From: Clément Bœsch
See https://github.com/gopro/gpmf-parser for more information on the
data stream itself.
---
libavformat/movenc.c | 25 +
1 file changed, 25 insertions(+)
diff --git a/libavformat/movenc.c b/libavformat/movenc.c
index 3989ac167e..0e5b45d150 100644
---
Le tridi 3 thermidor, an CCXXV, Zhong Li a écrit :
> Subject: Re: [FFmpeg-devel] [PATCH] doc/examples/encode_video: add qsv
> encoder support
I do not think it is a good idea. Examples are meant to be simple. I
think a separate example for hwdevice encoding would be a better idea,
although it has
Le tridi 3 thermidor, an CCXXV, Paul B Mahol a écrit :
> We will just ignore you as usual.
Will you please stop the ad-hominem attacks?
--
Nicolas George
signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg
2017-07-21 13:40 GMT+08:00 Zhong Li :
> Signed-off-by: Zhong Li
> ---
> doc/examples/encode_video.c | 32 +---
> 1 file changed, 29 insertions(+), 3 deletions(-)
>
> diff --git a/doc/examples/encode_video.c b/doc/examples/encode_video.c
> index 8cd1321..9c26f63 100644
65 matches
Mail list logo
