Re: [libav-devel] [PATCH] mpjpegdec: don't try to alloc an AVIOContext when probe is guaranteed to fail

On 08/06/15 22:19, James Almer wrote: The first check is done without the AVIOContext, so alloc it only if said check succeeds Signed-off-by: James Almer jamr...@gmail.com --- libavformat/mpjpegdec.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) Shouldn't hurt. lu

[libav-devel] [PATCH] mpjpegdec: don't try to alloc an AVIOContext when probe is guaranteed to fail

The first check is done without the AVIOContext, so alloc it only if said check succeeds Signed-off-by: James Almer jamr...@gmail.com --- libavformat/mpjpegdec.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavformat/mpjpegdec.c b/libavformat/mpjpegdec.c index

Re: [libav-devel] [PATCH] configure: Support MSVC 2015

On 08/06/15 09:20, Martin Storsjö wrote: On Sun, 7 Jun 2015, Luca Barbato wrote: The C runtime C99 compatibility had been improved a lot and it now rejects some of the compatibility defines provided for the older versions. Many thanks to Ray for the time spent testing. Bug-Id: 864 CC:

Re: [libav-devel] [PATCH] lavf: add a format flag for separate carriers, and an event for detecting carrier presence.

On 08/06/15 06:09, John Högberg wrote: Luca Barbato wrote: I was waiting for Anton, I'm more and more convinced that the feature is important and I'm planning to fit it in the avformat API that will match the avcodec API I detailed here[1]. I'm not sure about the name, but they can be

Re: [libav-devel] [PATCH] configure: Support MSVC 2015

On Sun, 7 Jun 2015, Luca Barbato wrote: The C runtime C99 compatibility had been improved a lot and it now rejects some of the compatibility defines provided for the older versions. Many thanks to Ray for the time spent testing. Bug-Id: 864 CC: libav-sta...@libav.org --- Martin does it sound

Re: [libav-devel] [PATCH] configure: Support MSVC 2015

On Mon, 8 Jun 2015, Luca Barbato wrote: On 08/06/15 09:20, Martin Storsjö wrote: On Sun, 7 Jun 2015, Luca Barbato wrote: The C runtime C99 compatibility had been improved a lot and it now rejects some of the compatibility defines provided for the older versions. Many thanks to Ray for the

Re: [libav-devel] [PATCH] lavf: add a format flag for separate carriers, and an event for detecting carrier presence.

Luca Barbato wrote: Please do. I thought about the alternative to support your scenario (beside having a callback API that I'll blog about soonish) and another way to do that is to have an AVSTREAM_EVENT_FLAG_LOST to signal that the stream is gone and an AVFMT_EVENT_FLAG_ACTIVE_STREAM_CHANGED

[libav-devel] mpegts: implement the CARRIER_PRESENT event

This patch implements the CARRIER_PRESENT event as outlined here; https://lists.libav.org/pipermail/libav-devel/2015-June/069889.html Regards, John Högberg From 163ff93fab1e0ec81f8f560d946dfe2a93e7c1d6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?John=20H=C3=B6gberg?= john.hogb...@ericsson.com Date:

[libav-devel] [PATCH 1/3] aac_parser: add required padding for GetBitContext buffer

Fixes stack buffer overflow errors detected by address sanitizer in various fate tests. --- libavcodec/aac_parser.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/aac_parser.c b/libavcodec/aac_parser.c index fdaa5f8..acb05d4 100644 --- a/libavcodec/aac_parser.c +++

[libav-devel] [PATCH 3/3] imc: add required padding for GetBitContext buffer

Fixes stack buffer overflow errors detected by address sanitizer in fate-imc. --- libavcodec/imc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/imc.c b/libavcodec/imc.c index 411bc85..c60fc7a 100644 --- a/libavcodec/imc.c +++ b/libavcodec/imc.c @@ -997,7 +997,7

[libav-devel] [PATCH 2/3] ac3_parser: add required padding for GetBitContext buffer

Fixes stack buffer overflow errors detected by address sanitizer in various fate tests. --- libavcodec/ac3_parser.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/ac3_parser.c b/libavcodec/ac3_parser.c index 5ea09f8..69d88c1 100644 --- a/libavcodec/ac3_parser.c +++

Re: [libav-devel] [PATCH 1/1] mpjpgdec: free AVIOContext leak on early probe fail

On 08/06/15 13:32, Janne Grunau wrote: --- libavformat/mpjpegdec.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/mpjpegdec.c b/libavformat/mpjpegdec.c index 354278c..72891e7 100644 --- a/libavformat/mpjpegdec.c +++ b/libavformat/mpjpegdec.c @@ -88,7

Re: [libav-devel] [PATCH 1/7] stcp: Always initialize outmsg struct

On 03/06/15 15:16, Vittorio Giovara wrote: CC: libav-sta...@libav.org Bug-Id: CID 1302711 --- libavformat/sctp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/sctp.c b/libavformat/sctp.c index 71b3a4b..57fd044 100644 --- a/libavformat/sctp.c +++

Re: [libav-devel] [PATCH 3/7] audiointerleave: Always initialize new_pkt

On 03/06/15 15:16, Vittorio Giovara wrote: CC: libav-sta...@libav.org Bug-Id: CID 609333 --- libavformat/audiointerleave.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/audiointerleave.c b/libavformat/audiointerleave.c index ba78d4e..fd07893 100644 ---

Re: [libav-devel] [PATCH 4/4] mpegvideo: Mark one function as static

On 08/06/15 14:00, Vittorio Giovara wrote: --- libavcodec/mpegvideo.c | 6 +++--- libavcodec/mpegvideo.h | 2 -- 2 files changed, 3 insertions(+), 5 deletions(-) Seems fine. lu ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH 1/2] mpegvideo: Split picture allocation for encoding and decoding

On 04/06/15 17:02, Vittorio Giovara wrote: The main ff_alloc_picture() function is made more generic with all the parameters necessary as arguments. This will allows to move most of the related functions to a separate file later. Right now wrappers are provided to try and minimize the number

Re: [libav-devel] [PATCH 2/2] mpegvideo: Move OutFormat enum to mpegutils.h

On 04/06/15 17:02, Vittorio Giovara wrote: It is necessary to avoid circular header dependencies. --- Header typo fixed. Vittorio As long oracle is happy, fine for me. ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH 5/7] mov: Check memory allocation

On 03/06/15 15:16, Vittorio Giovara wrote: CC: libav-sta...@libav.org Bug-Id: CID 1292518 --- libavformat/mov.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 80681b7..f603446 100644 --- a/libavformat/mov.c +++

Re: [libav-devel] [PATCH 3/4] mpegvideo: Move block permutation function where is used

On 08/06/15 14:00, Vittorio Giovara wrote: --- libavcodec/mpegvideo.c | 29 - libavcodec/mpegvideo.h | 6 -- libavcodec/mpegvideo_enc.c | 40 ++-- 3 files changed, 38 insertions(+), 37 deletions(-) Sure.

Re: [libav-devel] [PATCH 2/4] mpegvideo: Drop release_unused_pictures() function

On 08/06/15 14:00, Vittorio Giovara wrote: It is only used in one place, and it is small enough that it might be called directly. --- libavcodec/mpegvideo.c | 17 + 1 file changed, 5 insertions(+), 12 deletions(-) Ok. ___

[libav-devel] [PATCH 1/4] mpegvideo: Move Picture-related functions to a separate file

--- libavcodec/Makefile | 2 +- libavcodec/mpegpicture.c | 424 +++ libavcodec/mpegpicture.h | 109 libavcodec/mpegvideo.c | 397 +--- libavcodec/mpegvideo.h | 78 + 5 files

[libav-devel] [PATCH 1/1] mpjpgdec: free AVIOContext leak on early probe fail

--- libavformat/mpjpegdec.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/mpjpegdec.c b/libavformat/mpjpegdec.c index 354278c..72891e7 100644 --- a/libavformat/mpjpegdec.c +++ b/libavformat/mpjpegdec.c @@ -88,7 +88,7 @@ static int

Re: [libav-devel] [PATCH 4/7] jack: Check memory allocation

On 03/06/15 15:16, Vittorio Giovara wrote: CC: libav-sta...@libav.org Bug-Id: CID 1292520 --- libavdevice/jack.c | 4 1 file changed, 4 insertions(+) diff --git a/libavdevice/jack.c b/libavdevice/jack.c index 5d000a3..aa9348d 100644 --- a/libavdevice/jack.c +++

[libav-devel] [PATCH 3/4] mpegvideo: Move block permutation function where is used

--- libavcodec/mpegvideo.c | 29 - libavcodec/mpegvideo.h | 6 -- libavcodec/mpegvideo_enc.c | 40 ++-- 3 files changed, 38 insertions(+), 37 deletions(-) diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c

[libav-devel] [PATCH 4/4] mpegvideo: Mark one function as static

--- libavcodec/mpegvideo.c | 6 +++--- libavcodec/mpegvideo.h | 2 -- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c index 68f9568..0a4f5bb 100644 --- a/libavcodec/mpegvideo.c +++ b/libavcodec/mpegvideo.c @@ -1448,7 +1448,7 @@ void

[libav-devel] [PATCH 2/4] mpegvideo: Drop release_unused_pictures() function

It is only used in one place, and it is small enough that it might be called directly. --- libavcodec/mpegvideo.c | 17 + 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c index 86eab07..1ea2b65 100644 ---

Re: [libav-devel] [PATCH 1/4] mpegvideo: Move Picture-related functions to a separate file

On 08/06/15 14:00, Vittorio Giovara wrote: --- libavcodec/Makefile | 2 +- libavcodec/mpegpicture.c | 424 +++ libavcodec/mpegpicture.h | 109 libavcodec/mpegvideo.c | 397 +---

Re: [libav-devel] [PATCH 1/3] aac_parser: add required padding for GetBitContext buffer

On 08/06/15 15:09, Janne Grunau wrote: Fixes stack buffer overflow errors detected by address sanitizer in various fate tests. --- libavcodec/aac_parser.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/aac_parser.c b/libavcodec/aac_parser.c index