Re: [FFmpeg-devel] [PATCH] avcodec/h264_parse: Treat escaped and unescaped decoding error equal in decode_extradata_ps_mp4()

2017-12-15 Thread Alexander Agranovsky
Would it be possible to cherry-pick into n3.4? Thanks! On 12/2/17 20:02, Michael Niedermayer wrote: On Sat, Nov 25, 2017 at 11:08:46PM +0100, Clément Bœsch wrote: On Sat, Nov 25, 2017 at 10:49:09PM +0100, Michael Niedermayer wrote: Fixes: lorex.mp4 Signed-off-by: Michael Niedermayer ---

[FFmpeg-devel] [PATCH] avfilter/vf_drawtext: fixed default/flt formatting ignoring offset parameter

2016-09-26 Thread Alexander Agranovsky
From 9f183abbdde7fa50b9425165dd792d2770098749 Mon Sep 17 00:00:00 2001 From: Alex Agranovsky Date: Mon, 26 Sep 2016 16:23:45 -0400 Subject: [PATCH] avfilter/vf_drawtext: fixed default/flt formatting ignoring offset parameter Signed-off-by: Alex Agranovsky --- libavfilter/vf_drawtext.c | 2 +-

Re: [FFmpeg-devel] [PATCH 3/3] lavf/mpjpeg: do not include CRLF preceding boundary as, part of the returned frame

2016-02-14 Thread Alexander Agranovsky
Thanks - any problems with 2/3? Or is it still pending? On 2/14/16 8:53 AM, Michael Niedermayer wrote: On Sat, Feb 13, 2016 at 11:49:06PM -0500, Alexander Agranovsky wrote: mpjpegdec.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) 0c761f422ba83446d20728449c0c6813e4e11560

[FFmpeg-devel] [PATCH 3/3] lavf/mpjpeg: do not include CRLF preceding boundary as, part of the returned frame

2016-02-13 Thread Alexander Agranovsky
From b2ef061db32933e896dbeecb68268a62c47d6e0a Mon Sep 17 00:00:00 2001 From: Alex Agranovsky Date: Sat, 13 Feb 2016 23:16:45 -0500 Subject: [PATCH 3/3] lavf/mpjpeg: do not include CRLF preceding boundary as part of the returned frame Signed-off-by: Alex Agranovsky --- libavformat/mpjpegdec.c

[FFmpeg-devel] [PATCH 2/3] lavf/mpjpeg: probe should not depend on Content-Length

2016-02-13 Thread Alexander Agranovsky
From 83c0b5f7a64e4c47801ee2f94635eea5ea144eb1 Mon Sep 17 00:00:00 2001 From: Alex Agranovsky Date: Sat, 13 Feb 2016 23:15:20 -0500 Subject: [PATCH 2/3] lavf/mpjpeg: probe should not depend on Content-Length MIME header being present Signed-off-by: Alex Agranovsky --- libavformat/mpjpegdec.c

[FFmpeg-devel] [PATCH 1/3] lavf/mpjpeg: Trim quotes on MIME boundary, if present.

2016-02-13 Thread Alexander Agranovsky
From f2d8bc033cc6ab6e9a42472c99bdfce8305a13db Mon Sep 17 00:00:00 2001 From: Alex Agranovsky Date: Fri, 12 Feb 2016 12:59:29 -0500 Subject: [PATCH 1/3] lavf/mpjpeg: Trim quotes on MIME boundary, if present. Fixes 5023 Signed-off-by: Alex Agranovsky --- libavformat/mpjpegdec.c | 7 +++ 1

[FFmpeg-devel] [PATCH] lavf/mpjpegdec: Fixed dereference after null check

2015-12-09 Thread Alexander Agranovsky
From 36596b444e584eb7ccceb24c838f0273116691f8 Mon Sep 17 00:00:00 2001 From: Alex Agranovsky Date: Wed, 9 Dec 2015 12:48:20 -0500 Subject: [PATCH] lavf/mpjpegdec: Fixed dereference after null check Fixes Coverity CID 1341576 --- libavformat/mpjpegdec.c | 6 -- 1 file changed, 4 insertions(

Re: [FFmpeg-devel] [PATCH] Allow mpjpeg demuxer to process MIME parts which do not include Content-Length header.

2015-12-02 Thread Alexander Agranovsky
On 11/30/15 10:35 AM, Alexander Agranovsky wrote: On 11/30/15 10:24 AM, Nicolas George wrote: Le decadi 10 frimaire, an CCXXIV, Alexander Agranovsky a écrit : Without getting into a religious debate This is the reason I gave practical argument. As pointed later in the thread, lu is used

Re: [FFmpeg-devel] [PATCH] Allow mpjpeg demuxer to process MIME parts which do not include Content-Length header.

2015-11-30 Thread Alexander Agranovsky
On 11/30/15 10:24 AM, Nicolas George wrote: Le decadi 10 frimaire, an CCXXIV, Alexander Agranovsky a écrit : Without getting into a religious debate This is the reason I gave practical argument. As pointed later in the thread, lu is used elsewhere. And yes, MS makes it interesting in this

Re: [FFmpeg-devel] [PATCH] Allow mpjpeg demuxer to process MIME parts which do not include Content-Length header.

2015-11-30 Thread Alexander Agranovsky
On 11/30/15 7:34 AM, wm4 wrote: On Mon, 30 Nov 2015 07:27:14 -0500 Alexander Agranovsky wrote: On 11/30/15 2:41 AM, Nicolas George wrote: Le nonidi 9 frimaire, an CCXXIV, Alexander Agranovsky a écrit : Please see the updated patches attached. The trimming loop that was subject of the

Re: [FFmpeg-devel] [PATCH] Allow mpjpeg demuxer to process MIME parts which do not include Content-Length header.

2015-11-30 Thread Alexander Agranovsky
On 11/30/15 7:34 AM, wm4 wrote: On Mon, 30 Nov 2015 07:27:14 -0500 Alexander Agranovsky wrote: On 11/30/15 2:41 AM, Nicolas George wrote: Le nonidi 9 frimaire, an CCXXIV, Alexander Agranovsky a écrit : Please see the updated patches attached. The trimming loop that was subject of the

Re: [FFmpeg-devel] [PATCH] Allow mpjpeg demuxer to process MIME parts which do not include Content-Length header.

2015-11-30 Thread Alexander Agranovsky
On 11/30/15 6:37 AM, wm4 wrote: On Mon, 30 Nov 2015 08:41:42 +0100 Nicolas George wrote: +"Expected boundary '%s' not found, instead found a line of %lu bytes\n", +expected_boundary, +strlen(line)); "%lu" is not correct for size_t. The correc

Re: [FFmpeg-devel] [PATCH] Allow mpjpeg demuxer to process MIME parts which do not include Content-Length header.

2015-11-30 Thread Alexander Agranovsky
On 11/30/15 2:41 AM, Nicolas George wrote: Le nonidi 9 frimaire, an CCXXIV, Alexander Agranovsky a écrit : Please see the updated patches attached. The trimming loop that was subject of the discussion had been rewritten to use indices rather than pointer arithmetics. This kind of drastic

Re: [FFmpeg-devel] [PATCH] Allow mpjpeg demuxer to process MIME parts which do not include Content-Length header.

2015-11-29 Thread Alexander Agranovsky
On 11/29/15 1:16 PM, Nicolas George wrote: Le nonidi 9 frimaire, an CCXXIV, Ganesh Ajjanagadde a écrit : end = p + strlen(p) - 1; Don't know what you are referring to here, but dereferencing is clearly invalid. However, in order to allow common loop idioms, pointer arithmetic one elemen

Re: [FFmpeg-devel] [PATCH] Allow mpjpeg demuxer to process MIME parts which do not include Content-Length header.

2015-11-29 Thread Alexander Agranovsky
On 11/29/15 1:00 PM, wm4 wrote: On Wed, 25 Nov 2015 10:35:31 -0500 Alex Agranovsky wrote: From 4797590e11267993c3883e5037619625e1f1dadf Mon Sep 17 00:00:00 2001 From: Alex Agranovsky Date: Tue, 24 Nov 2015 00:07:34 -0500 Subject: [PATCH 2/2] If available, use the actual boundary in HTTP re