Re: [libav-devel] [PATCH] Announce Release 10.3

2014-08-03 Thread Diego Biurrun
On Sun, Aug 03, 2014 at 10:55:21PM -0400, Reinhard Tartler wrote: > --- > src/download | 22 +++--- > src/news | 28 > 2 files changed, 39 insertions(+), 11 deletions(-) LGTM Diego ___ libav-devel mailin

Re: [libav-devel] [PATCH] [RFC] imgutils: Do not declare avpriv_set_systematic_pal2 in the public header

2014-08-03 Thread Martin Storsjö
On Mon, 4 Aug 2014, Vittorio Giovara wrote: On Sun, Aug 3, 2014 at 9:34 PM, Diego Biurrun wrote: --- This probably requires some sort of version adjustment or so... libavutil/imgutils.h | 2 -- libavutil/internal.h | 3 +++ 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/liba

Re: [libav-devel] [PATCH] libavformat/icecast: Add Icecast protocol

2014-08-03 Thread Martin Storsjö
On Sun, 3 Aug 2014, Diego Biurrun wrote: On Mon, Jul 28, 2014 at 11:55:38AM +0200, epira...@gmail.com wrote: From: Marvin Scholz +// Check for auth data in URI +if (auth[0]) { +char *sep = strchr(auth,':'); +if (sep) { +*sep = 0; +sep++; +

[libav-devel] [PATCH] Announce Release 10.3

2014-08-03 Thread Reinhard Tartler
--- src/download | 22 +++--- src/news | 28 2 files changed, 39 insertions(+), 11 deletions(-) diff --git a/src/download b/src/download index 83cc266..67463e3 100644 --- a/src/download +++ b/src/download @@ -201,23 +201,23 @@ and much faster bug f

Re: [libav-devel] [PATCH] mpegts: Define the section lenght with a constant

2014-08-03 Thread Timothy Gu
On Aug 3, 2014 2:42 PM, "Luca Barbato" wrote: > > The specification says the value is expressed in 10bit including, > the 4bytes crc. > --- > > Now with the correct value, hopefully. > > libavformat/mpegtsenc.c | 10 +++--- > 1 file changed, 7 insertions(+), 3 deletions(-) > > diff --git a/li

[libav-devel] [PATCH] [9] [0.8] vp3: Copy all 3 frames for thread updates

2014-08-03 Thread Vittorio Giovara
From: Michael Niedermayer Fixes a double release of the current frame on deinit. Bug-Id: CVE-2011-3934 Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind Signed-off-by: Vittorio Giovara --- Better commit title suggestions are welcome. Vittorio libavcodec/vp3.c | 2 +- 1 file changed, 1 in

Re: [libav-devel] [PATCH] [RFC] imgutils: Do not declare avpriv_set_systematic_pal2 in the public header

2014-08-03 Thread Vittorio Giovara
On Sun, Aug 3, 2014 at 9:34 PM, Diego Biurrun wrote: > --- > > This probably requires some sort of version adjustment or so... > > libavutil/imgutils.h | 2 -- > libavutil/internal.h | 3 +++ > 2 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/libavutil/imgutils.h b/libavutil/imgu

[libav-devel] [PATCH 1/2] fate: Only generate tests/pixfmts.mak if some pixfmts fate test is run

2014-08-03 Thread Diego Biurrun
--- tests/fate/filter-video.mak | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/fate/filter-video.mak b/tests/fate/filter-video.mak index bbe77ce..d1cb721 100644 --- a/tests/fate/filter-video.mak +++ b/tests/fate/filter-video.mak @@ -122,7 +122,9 @@ tests/pixfmts.mak:

[libav-devel] [PATCH 2/2] build: Add explicit dependency declaration for generated version.h header

2014-08-03 Thread Diego Biurrun
Otherwise the dependency is noticed only after compiling the first file that requires it, resulting in double compilation of that file. --- Makefile | 1 + 1 file changed, 1 insertion(+) diff --git a/Makefile b/Makefile index cc016b3..8a484c5 100644 --- a/Makefile +++ b/Makefile @@ -162,6 +162,7

Re: [libav-devel] [PATCH] avcodec: Deprecate dtg_active_format field in favor of avframe side-data

2014-08-03 Thread Luca Barbato
On 03/08/14 23:06, Diego Biurrun wrote: > From: Kieran Kunhya > > Signed-off-by: Diego Biurrun > --- > > FATE passes, log message updated. > Possibly Ok. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/li

Re: [libav-devel] [PATCH] libavformat/icecast: Add Icecast protocol

2014-08-03 Thread Diego Biurrun
On Mon, Jul 28, 2014 at 11:55:38AM +0200, epira...@gmail.com wrote: > From: Marvin Scholz > > Check if strdup failed > --- > Icecast is basically a convenience wrapper around the HTTP protocol. > --- Add your annotations below, not above, the ---, otherwise they replace the log message. > --- /

[libav-devel] [PATCH] mpegts: Define the section lenght with a constant

2014-08-03 Thread Luca Barbato
The specification says the value is expressed in 10bit including, the 4bytes crc. --- Now with the correct value, hopefully. libavformat/mpegtsenc.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c index 838702e..b29

[libav-devel] [PATCH] avcodec: Deprecate dtg_active_format field in favor of avframe side-data

2014-08-03 Thread Diego Biurrun
From: Kieran Kunhya Signed-off-by: Diego Biurrun --- FATE passes, log message updated. doc/APIchanges| 4 libavcodec/avcodec.h | 5 - libavcodec/mpeg12dec.c| 20 +++- libavcodec/version.h | 5 - libavfilter/vf_showinfo.c | 3 +++ liba

Re: [libav-devel] [PATCH] Deprecate AFD field and add AFD as side-data

2014-08-03 Thread Diego Biurrun
On Sun, Aug 03, 2014 at 09:53:16PM +0100, Kieran Kunhya wrote: > > +sd->data= s1->afd; > > probably the crash is here. Yes, *sd->data and it works. Diego ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman

Re: [libav-devel] [PATCH] [RFC] imgutils: Do not declare avpriv_set_systematic_pal2 in the public header

2014-08-03 Thread Luca Barbato
On 03/08/14 22:34, Diego Biurrun wrote: > --- > > This probably requires some sort of version adjustment or so... > > libavutil/imgutils.h | 2 -- > libavutil/internal.h | 3 +++ > 2 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/libavutil/imgutils.h b/libavutil/imgutils.h > in

Re: [libav-devel] [PATCH] Deprecate AFD field and add AFD as side-data

2014-08-03 Thread Kieran Kunhya
> +sd->data= s1->afd; probably the crash is here. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH] mpegts: Define the section data lenght with a constant

2014-08-03 Thread Luca Barbato
--- libavformat/mpegtsenc.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c index 838702e..571b9bf 100644 --- a/libavformat/mpegtsenc.c +++ b/libavformat/mpegtsenc.c @@ -89,6 +89,8 @@ typedef struct MpegTSWrite { #defin

[libav-devel] [PATCH] [RFC] imgutils: Do not declare avpriv_set_systematic_pal2 in the public header

2014-08-03 Thread Diego Biurrun
--- This probably requires some sort of version adjustment or so... libavutil/imgutils.h | 2 -- libavutil/internal.h | 3 +++ 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/libavutil/imgutils.h b/libavutil/imgutils.h index 313037a..50fa815 100644 --- a/libavutil/imgutils.h +++ b

Re: [libav-devel] [PATCH] Deprecate AFD field and add AFD as side-data

2014-08-03 Thread Diego Biurrun
On Sun, Aug 03, 2014 at 01:00:03PM -0700, Diego Biurrun wrote: > From: Kieran Kunhya > > Signed-off-by: Diego Biurrun > --- > > Fixed some small stylistic issues. > > doc/APIchanges| 4 > libavcodec/avcodec.h | 5 - > libavcodec/mpeg12dec.c| 20

Re: [libav-devel] [PATCH 5/5] huffyuv: Check and propagate function return values

2014-08-03 Thread Vittorio Giovara
On Sun, Aug 3, 2014 at 8:36 PM, Diego Biurrun wrote: > Bug-Id: CVE-2013-0868 > > CC: libav-sta...@libav.org > --- > This and the set look good. Vittorio ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-dev

Re: [libav-devel] [PATCH 4/5] huffyuv: Return proper error codes

2014-08-03 Thread Luca Barbato
On 03/08/14 21:36, Diego Biurrun wrote: > --- > libavcodec/huffyuvdec.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > Ok. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 3/5] huffyuv: Use avpriv_report_missing_feature() where appropriate

2014-08-03 Thread Luca Barbato
On 03/08/14 21:36, Diego Biurrun wrote: > --- > libavcodec/huffyuvdec.c | 10 -- > 1 file changed, 4 insertions(+), 6 deletions(-) > Ok. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 2/5] huffyuv: Eliminate some pointless casts

2014-08-03 Thread Luca Barbato
On 03/08/14 21:36, Diego Biurrun wrote: > --- > libavcodec/huffyuvdec.c | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) > Ok. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 1/5] huffyuv: K&R formatting cosmetics

2014-08-03 Thread Luca Barbato
On 03/08/14 21:36, Diego Biurrun wrote: > --- > libavcodec/huffyuvdec.c | 334 > ++-- > 1 file changed, 178 insertions(+), 156 deletions(-) > Ok. ___ libav-devel mailing list libav-devel@libav.org https://l

Re: [libav-devel] [PATCH] fate: png-suite

2014-08-03 Thread Luca Barbato
On 03/08/14 21:40, Vittorio Giovara wrote: > Every supported format is converted to RGB. > --- > This version makes all oracle happy apparently. > Vittorio > > tests/fate/image.mak | 12 > tests/ref/fate/png-gray16 | 2 ++ > tests/ref/fate/png-gray8 | 2 ++ > tests/ref/fate/p

[libav-devel] [PATCH] Deprecate AFD field and add AFD as side-data

2014-08-03 Thread Diego Biurrun
From: Kieran Kunhya Signed-off-by: Diego Biurrun --- Fixed some small stylistic issues. doc/APIchanges| 4 libavcodec/avcodec.h | 5 - libavcodec/mpeg12dec.c| 20 +++- libavcodec/version.h | 5 - libavfilter/vf_showinfo.c | 3 +++ lib

[libav-devel] [PATCH] fate: png-suite

2014-08-03 Thread Vittorio Giovara
Every supported format is converted to RGB. --- This version makes all oracle happy apparently. Vittorio tests/fate/image.mak | 12 tests/ref/fate/png-gray16 | 2 ++ tests/ref/fate/png-gray8 | 2 ++ tests/ref/fate/png-rgb24 | 2 ++ tests/ref/fate/png-rgb48 | 2 ++ tests/r

[libav-devel] [PATCH 3/5] huffyuv: Use avpriv_report_missing_feature() where appropriate

2014-08-03 Thread Diego Biurrun
--- libavcodec/huffyuvdec.c | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/libavcodec/huffyuvdec.c b/libavcodec/huffyuvdec.c index b2fc3aa..ebde3ca 100644 --- a/libavcodec/huffyuvdec.c +++ b/libavcodec/huffyuvdec.c @@ -530,9 +530,8 @@ static int decode_frame(AVCodec

[libav-devel] [PATCH 4/5] huffyuv: Return proper error codes

2014-08-03 Thread Diego Biurrun
--- libavcodec/huffyuvdec.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/huffyuvdec.c b/libavcodec/huffyuvdec.c index ebde3ca..4d62ab3 100644 --- a/libavcodec/huffyuvdec.c +++ b/libavcodec/huffyuvdec.c @@ -98,7 +98,7 @@ static int read_len_table(uint8_t *dst

[libav-devel] [PATCH 1/5] huffyuv: K&R formatting cosmetics

2014-08-03 Thread Diego Biurrun
--- libavcodec/huffyuvdec.c | 334 ++-- 1 file changed, 178 insertions(+), 156 deletions(-) diff --git a/libavcodec/huffyuvdec.c b/libavcodec/huffyuvdec.c index e0d8a06..e57b461 100644 --- a/libavcodec/huffyuvdec.c +++ b/libavcodec/huffyuvdec.c @@ -36,5

[libav-devel] [PATCH 2/5] huffyuv: Eliminate some pointless casts

2014-08-03 Thread Diego Biurrun
--- libavcodec/huffyuvdec.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/libavcodec/huffyuvdec.c b/libavcodec/huffyuvdec.c index e57b461..b2fc3aa 100644 --- a/libavcodec/huffyuvdec.c +++ b/libavcodec/huffyuvdec.c @@ -263,17 +263,17 @@ static av_cold int decode_i

[libav-devel] [PATCH 5/5] huffyuv: Check and propagate function return values

2014-08-03 Thread Diego Biurrun
Bug-Id: CVE-2013-0868 CC: libav-sta...@libav.org --- This is inspired by Michael's patch, but I redid it completely from scratch as it was woefully incomplete anyway... libavcodec/huffyuvdec.c | 105 1 file changed, 61 insertions(+), 44 deletions

Re: [libav-devel] [PATCH] huffyuvdec: propagate init_vlc() return error

2014-08-03 Thread Diego Biurrun
On Sun, Aug 03, 2014 at 06:55:48PM +0200, Diego Biurrun wrote: > On Sun, Aug 03, 2014 at 01:14:07AM +0100, Vittorio Giovara wrote: > > --- a/libavcodec/huffyuvdec.c > > +++ b/libavcodec/huffyuvdec.c > > @@ -190,13 +195,15 @@ static int read_huffman_tables(HYuvContext *s, const > > uint8_t *src, in

Re: [libav-devel] [PATCH] [9] cmdutil: Add image size check to codec_get_buffer()

2014-08-03 Thread Justin Ruggles
On 08/03/2014 11:32 AM, Diego Biurrun wrote: From: Michael Niedermayer Bug-Id: CVE-2011-3935 Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer Signed-off-by: Diego Biurrun --- Applies to the 9 branch; again, no sample. cmdutils.c | 3 +++ 1 file c

Re: [libav-devel] [PATCHv3] Deprecate AFD field and add AFD as side-data

2014-08-03 Thread Vittorio Giovara
On Sun, Aug 3, 2014 at 7:24 PM, Kieran Kunhya wrote: > --- > doc/APIchanges|4 > libavcodec/avcodec.h |5 - > libavcodec/mpeg12dec.c| 20 +++- > libavcodec/version.h |5 - > libavfilter/vf_showinfo.c |3 +++ > libavutil/fram

Re: [libav-devel] [PATCH 1/2] fate: add informative cpu test

2014-08-03 Thread Diego Biurrun
On Sun, Aug 03, 2014 at 08:16:18PM +0200, Janne Grunau wrote: > On 2014-08-03 18:06:04 +0200, Diego Biurrun wrote: > > On Sun, Aug 03, 2014 at 04:37:05PM +0200, Janne Grunau wrote: > > > --- a/libavutil/cpu.c > > > +++ b/libavutil/cpu.c > > > @@ -224,17 +228,50 @@ static const struct { > > > > >

Re: [libav-devel] [PATCH] mpeg4video: Initialize xvididct for all threads

2014-08-03 Thread Diego Biurrun
On Sun, Aug 03, 2014 at 08:12:17PM +0200, Janne Grunau wrote: > On 2014-08-03 09:02:03 -0700, Diego Biurrun wrote: > > --- a/libavcodec/mpeg4videodec.c > > +++ b/libavcodec/mpeg4videodec.c > > @@ -2539,14 +2539,19 @@ static int > > mpeg4_update_thread_context(AVCodecContext *dst, > > { > > M

Re: [libav-devel] [PATCHv3] Deprecate AFD field and add AFD as side-data

2014-08-03 Thread Luca Barbato
On 03/08/14 20:24, Kieran Kunhya wrote: > --- > doc/APIchanges|4 > libavcodec/avcodec.h |5 - > libavcodec/mpeg12dec.c| 20 +++- > libavcodec/version.h |5 - > libavfilter/vf_showinfo.c |3 +++ > libavutil/frame.h |

[libav-devel] [PATCH] mmvideo: check horizontal coordinate too

2014-08-03 Thread Vittorio Giovara
From: Michael Niedermayer Fixes out of array accesses. Bug-Id: CVE-2013-3672 Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind Signed-off-by: Vittorio Giovara --- libavcodec/mmvideo.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/mmvideo.c b/libavcodec/mmvideo.c index a

[libav-devel] [PATCHv3] Deprecate AFD field and add AFD as side-data

2014-08-03 Thread Kieran Kunhya
--- doc/APIchanges|4 libavcodec/avcodec.h |5 - libavcodec/mpeg12dec.c| 20 +++- libavcodec/version.h |5 - libavfilter/vf_showinfo.c |3 +++ libavutil/frame.h | 16 libavutil/version.h |

Re: [libav-devel] [PATCH 1/2] fate: add informative cpu test

2014-08-03 Thread Janne Grunau
On 2014-08-03 18:06:04 +0200, Diego Biurrun wrote: > On Sun, Aug 03, 2014 at 04:37:05PM +0200, Janne Grunau wrote: > > --- a/libavutil/cpu.c > > +++ b/libavutil/cpu.c > > @@ -224,17 +228,50 @@ static const struct { > > > > +static void print_cpu_flags(int cpu_flags, const char *type) > > { > >

Re: [libav-devel] [PATCH] mpeg4video: Initialize xvididct for all threads

2014-08-03 Thread Janne Grunau
On 2014-08-03 09:02:03 -0700, Diego Biurrun wrote: > From: Anton Khirnov > > Signed-off-by: Diego Biurrun > --- > > This fixes fate-xvid-idctmmx with more than one thread. > > libavcodec/mpeg4videodec.c | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/libavcodec/mpeg4videodec.c

Re: [libav-devel] [PATCH] huffyuvdec: propagate init_vlc() return error

2014-08-03 Thread Diego Biurrun
On Sun, Aug 03, 2014 at 01:14:07AM +0100, Vittorio Giovara wrote: > --- a/libavcodec/huffyuvdec.c > +++ b/libavcodec/huffyuvdec.c > @@ -190,13 +195,15 @@ static int read_huffman_tables(HYuvContext *s, const > uint8_t *src, int length) > ff_free_vlc(&s->vlc[i]); > -init_vlc(&s->vlc

Re: [libav-devel] [PATCH 1/1] aarch64: use MACH-O const data asm directive in const macro

2014-08-03 Thread Martin Storsjö
On Sun, 3 Aug 2014, Janne Grunau wrote: --- libavutil/aarch64/asm.S | 4 1 file changed, 4 insertions(+) diff --git a/libavutil/aarch64/asm.S b/libavutil/aarch64/asm.S index 850f16b..b766e67 100644 --- a/libavutil/aarch64/asm.S +++ b/libavutil/aarch64/asm.S @@ -57,7 +57,11 @@ FUNC.func

Re: [libav-devel] [PATCH] [9] cmdutil: Add image size check to codec_get_buffer()

2014-08-03 Thread Luca Barbato
On 03/08/14 17:32, Diego Biurrun wrote: > From: Michael Niedermayer > > Bug-Id: CVE-2011-3935 > > Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind > Signed-off-by: Michael Niedermayer > Signed-off-by: Diego Biurrun > --- > > Applies to the 9 branch; again, no sample. > > cmdutils.c |

Re: [libav-devel] [PATCH 1/2] fate: add informative cpu test

2014-08-03 Thread Diego Biurrun
On Sun, Aug 03, 2014 at 04:37:05PM +0200, Janne Grunau wrote: > --- a/libavutil/cpu.c > +++ b/libavutil/cpu.c > @@ -224,17 +228,50 @@ static const struct { > > +static void print_cpu_flags(int cpu_flags, const char *type) > { > int i; > > +fprintf(stderr, "cpu_flags(%s) = 0x%08X\n", t

Re: [libav-devel] [PATCH] [0.8] avconv: add image size check to codec_get_buffer()

2014-08-03 Thread Luca Barbato
On 03/08/14 17:29, Diego Biurrun wrote: > From: Michael Niedermayer > > Fixes CVE-2011-3935 > > Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind > Signed-off-by: Michael Niedermayer > Signed-off-by: Diego Biurrun > --- > > This is the version for the 0.8 branch. I have no sample to ver

[libav-devel] [PATCH] mpeg4video: Initialize xvididct for all threads

2014-08-03 Thread Diego Biurrun
From: Anton Khirnov Signed-off-by: Diego Biurrun --- This fixes fate-xvid-idctmmx with more than one thread. libavcodec/mpeg4videodec.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavcodec/mpeg4videodec.c b/libavcodec/mpeg4videodec.c index 25334ac..c17e385 100644 --- a/libavcod

Re: [libav-devel] [PATCH 1/2] fate: add informative cpu test

2014-08-03 Thread Luca Barbato
On 03/08/14 16:37, Janne Grunau wrote: > libavutil/cpu-test prints raw and effective cpu flags to STDERR. Detected > cpu flags can be useful for debugging fate errors. > > No comparison of the result against a expected result since that would > require fate config specific references. > --- > lib

Re: [libav-devel] [PATCH 1/1] aarch64: use MACH-O const data asm directive in const macro

2014-08-03 Thread Luca Barbato
On 03/08/14 17:09, Janne Grunau wrote: > --- > libavutil/aarch64/asm.S | 4 > 1 file changed, 4 insertions(+) > > diff --git a/libavutil/aarch64/asm.S b/libavutil/aarch64/asm.S > index 850f16b..b766e67 100644 > --- a/libavutil/aarch64/asm.S > +++ b/libavutil/aarch64/asm.S > @@ -57,7 +57,11 @

[libav-devel] [PATCH] [9] cmdutil: Add image size check to codec_get_buffer()

2014-08-03 Thread Diego Biurrun
From: Michael Niedermayer Bug-Id: CVE-2011-3935 Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer Signed-off-by: Diego Biurrun --- Applies to the 9 branch; again, no sample. cmdutils.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/cmdutils.c

[libav-devel] [PATCH] [0.8] avconv: add image size check to codec_get_buffer()

2014-08-03 Thread Diego Biurrun
From: Michael Niedermayer Fixes CVE-2011-3935 Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer Signed-off-by: Diego Biurrun --- This is the version for the 0.8 branch. I have no sample to verify the fix. avconv.c | 3 +++ 1 file changed, 3 insertion

[libav-devel] [PATCH 1/1] aarch64: use MACH-O const data asm directive in const macro

2014-08-03 Thread Janne Grunau
--- libavutil/aarch64/asm.S | 4 1 file changed, 4 insertions(+) diff --git a/libavutil/aarch64/asm.S b/libavutil/aarch64/asm.S index 850f16b..b766e67 100644 --- a/libavutil/aarch64/asm.S +++ b/libavutil/aarch64/asm.S @@ -57,7 +57,11 @@ FUNC.func \name ELF .size \name, . - \name

[libav-devel] [PATCH 2/2] cpu-test: test av_cpu_count

2014-08-03 Thread Janne Grunau
Add CPU count and number threads as informative values for fate. --- libavutil/cpu.c | 14 +- tests/fate/libavutil.mak | 2 +- 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/libavutil/cpu.c b/libavutil/cpu.c index 20f0fe1..a7e5f7f 100644 --- a/libavutil/cpu.c

[libav-devel] [PATCH 1/2] fate: add informative cpu test

2014-08-03 Thread Janne Grunau
libavutil/cpu-test prints raw and effective cpu flags to STDERR. Detected cpu flags can be useful for debugging fate errors. No comparison of the result against a expected result since that would require fate config specific references. --- libavutil/cpu.c | 49 ++

Re: [libav-devel] [PATCH] svq1: do not modify the input packet

2014-08-03 Thread Diego Biurrun
On Sun, Aug 03, 2014 at 02:35:35PM +0200, Diego Biurrun wrote: > --- a/libavcodec/svq1dec.c > +++ b/libavcodec/svq1dec.c > @@ -60,6 +60,10 @@ typedef struct SVQ1Context { > AVFrame *prev; > + > +uint8_t *pkt_swapped; > +int pkt_swapped_allocated; > + > int width; > @@ -626,7 +630,

[libav-devel] [PATCH] svq1: do not modify the input packet

2014-08-03 Thread Diego Biurrun
From: Anton Khirnov The input data must remain constant, make a copy instead. This is in theory a performance hit, but since I failed to find any samples using this feature, this should not matter in practice. Also, check the size of the header, avoiding invalid reads on truncated data. CC:liba

Re: [libav-devel] [PATCH] http: Refactor http_open_cnx

2014-08-03 Thread Luca Barbato
On 03/08/14 11:28, Martin Storsjö wrote: > On Sat, 2 Aug 2014, Luca Barbato wrote: > >> Split return value handling from the actual opening. >> >> Incidentally fixes the https -> http redirect issue reported by >> Compn on behalf of rcombs. >> >> CC: libav-sta...@libav.org >> --- >> libavformat/ht

Re: [libav-devel] [PATCH 1/2] aarch64: use MACH-O const data asm directive in const macro

2014-08-03 Thread Janne Grunau
On 2014-08-03 12:23:35 +0300, Martin Storsjö wrote: > On Sun, 3 Aug 2014, Martin Storsjö wrote: > > >On Thu, 24 Jul 2014, Janne Grunau wrote: > > > >>--- > >>libavutil/aarch64/asm.S | 9 - > >>1 file changed, 8 insertions(+), 1 deletion(-) > >> > >>diff --git a/libavutil/aarch64/asm.S b/lib

Re: [libav-devel] [PATCH] swscale: prevent an invalid write of 1 byte

2014-08-03 Thread Kostya Shishkov
On Sun, Aug 03, 2014 at 10:35:35AM +0100, Vittorio Giovara wrote: > Bug-Id: 772 > CC: libav-sta...@libav.org > Found-By: Justin Ruggles > --- > libswscale/swscale_unscaled.c | 4 > 1 file changed, 4 deletions(-) > > diff --git a/libswscale/swscale_unscaled.c b/libswscale/swscale_unscaled.c

Re: [libav-devel] [PATCH] cdgraphics: check buffer size before use

2014-08-03 Thread Vittorio Giovara
On Sun, Aug 3, 2014 at 1:24 AM, Luca Barbato wrote: > On 03/08/14 02:17, Vittorio Giovara wrote: >> buf += 2; /// skipping 2 unneeded bytes >> -bytestream_get_buffer(&buf, cdg_data, buf_size - CDG_HEADER_SIZE); >> + >> +if (buf_size > CDG_HEADER_SIZE) >> +bytestream_get_buffe

Re: [libav-devel] [PATCH] libavformat/icecast: Add Icecast protocol

2014-08-03 Thread Martin Storsjö
On Mon, 28 Jul 2014, epira...@gmail.com wrote: From: Marvin Scholz Check if strdup failed --- Icecast is basically a convenience wrapper around the HTTP protocol. --- Changelog| 1 + configure| 1 + doc/general.texi | 1 + doc/protocols.texi |

[libav-devel] [PATCH] swscale: prevent an invalid write of 1 byte

2014-08-03 Thread Vittorio Giovara
Bug-Id: 772 CC: libav-sta...@libav.org Found-By: Justin Ruggles --- libswscale/swscale_unscaled.c | 4 1 file changed, 4 deletions(-) diff --git a/libswscale/swscale_unscaled.c b/libswscale/swscale_unscaled.c index 2907918..9ef456e 100644 --- a/libswscale/swscale_unscaled.c +++ b/libswscale

Re: [libav-devel] [PATCH] http: Refactor http_open_cnx

2014-08-03 Thread Martin Storsjö
On Sat, 2 Aug 2014, Luca Barbato wrote: Split return value handling from the actual opening. Incidentally fixes the https -> http redirect issue reported by Compn on behalf of rcombs. CC: libav-sta...@libav.org --- libavformat/http.c | 40 ++-- 1 file changed

Re: [libav-devel] [PATCH 1/2] aarch64: use MACH-O const data asm directive in const macro

2014-08-03 Thread Martin Storsjö
On Sun, 3 Aug 2014, Martin Storsjö wrote: On Thu, 24 Jul 2014, Janne Grunau wrote: --- libavutil/aarch64/asm.S | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/libavutil/aarch64/asm.S b/libavutil/aarch64/asm.S index 850f16b..f765e59 100644 --- a/libavutil/aarch64/asm.

Re: [libav-devel] [PATCH 2/2] aarch64: add ', ' between assembler macro arguments where missing

2014-08-03 Thread Martin Storsjö
On Thu, 24 Jul 2014, Janne Grunau wrote: llvm's integrated assembler does not accept spaces as macro argument delimiter when targeting darwin. Using a explicit delimiter is a good idea in principle since it makes case like 'macro 4 -2' vs 'macro 4 - 2' clear. --- libavcodec/aarch64/fft_neon.S

Re: [libav-devel] [PATCH 1/2] aarch64: use MACH-O const data asm directive in const macro

2014-08-03 Thread Martin Storsjö
On Thu, 24 Jul 2014, Janne Grunau wrote: --- libavutil/aarch64/asm.S | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/libavutil/aarch64/asm.S b/libavutil/aarch64/asm.S index 850f16b..f765e59 100644 --- a/libavutil/aarch64/asm.S +++ b/libavutil/aarch64/asm.S @@ -26,6 +26

Re: [libav-devel] [PATCH 2/2] aarch64: add ', ' between assembler macro arguments where missing

2014-08-03 Thread Diego Biurrun
On Thu, Jul 24, 2014 at 03:11:26PM +0200, Janne Grunau wrote: > llvm's integrated assembler does not accept spaces as macro argument > delimiter when targeting darwin. Using a explicit delimiter is a good > idea in principle since it makes case like 'macro 4 -2' vs 'macro 4 - 2' > clear. > --- > l

Re: [libav-devel] AArch64 asm fixes for llvm 3.5's integrated assembler on darwin

2014-08-03 Thread Janne Grunau
On 2014-07-24 15:11:24 +0200, Janne Grunau wrote: > Hi, > > following two patches improve support for llvm 3.5's integrated assembler > on darwin/ios. A locally patched llvm-3.5 with http://reviews.llvm.org/D3507 > passes make fate. ping Janne ___ liba

Re: [libav-devel] [PATCH] avcodec: Deprecate unused defines and options

2014-08-03 Thread Diego Biurrun
On Sat, Aug 02, 2014 at 01:00:33PM -0700, Diego Biurrun wrote: > --- > > Now leaves the members used to deliver encoding statistics alone. > > doc/APIchanges | 7 +++ > libavcodec/avcodec.h | 12 +++- > libavcodec/options_table.h | 10 ++ > libavcodec/versi

Re: [libav-devel] [PATCH 3/4] avcodec: options: Add missing deprecation ifdefs around emu_edge

2014-08-03 Thread Diego Biurrun
On Sat, Aug 02, 2014 at 10:12:57PM +0100, Vittorio Giovara wrote: > On Sat, Aug 2, 2014 at 8:42 PM, Diego Biurrun wrote: > > --- > > libavcodec/options_table.h | 2 ++ > > 1 file changed, 2 insertions(+) > > I did found that one too (in http://patchwork.libav.org/patch/48786/) > but thought that

Re: [libav-devel] [PATCH] lcl: Disentangle pointers to input data and decompression buffer

2014-08-03 Thread Diego Biurrun
On Sat, Aug 02, 2014 at 06:20:05AM -0700, Diego Biurrun wrote: > This is cleaner and avoids a cast plus a related const qualifier warning. > --- > > Redone according to Kostya's suggestion. > > libavcodec/lcldec.c | 25 - > 1 file changed, 12 insertions(+), 13 deletions(-