Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpjpegdec: fix strict boundary search string

2019-05-26 Thread Moritz Barsnick
On Sun, May 26, 2019 at 22:44:00 +0200, Moritz Barsnick wrote: > Subject: [FFmpeg-devel] [PATCH 2/2] avformat/mpjpegdec: fix strict boundary > search string D'uh, I told git send-email that the subject should contain "V2". I wonder why that gets

[FFmpeg-devel] [PATCH 2/2] avformat/mpjpegdec: fix strict boundary search string

2019-05-26 Thread Moritz Barsnick
According to RFC1341, the multipart boundary indicated by the Content-Type header must be prepended by CRLF + "--", and followed by CRLF. In the case of strict MIME header boundary handling, the "--" was forgotten to add. Fixes trac #7921. Signed-off-by: Moritz Barsnick ---

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpjpegdec: fix strict boundary search string

2019-05-26 Thread Moritz Barsnick
On Fri, May 24, 2019 at 11:58:51 +0200, Michael Niedermayer wrote: > > if (boundary != NULL) { > > -mpjpeg->boundary = boundary; > > -mpjpeg->searchstr = av_asprintf( "\r\n%s\r\n", boundary ); > > +mpjpeg->boundary = av_asprintf("--%s", boundary); > >

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpjpegdec: fix strict boundary search string

2019-05-24 Thread Michael Niedermayer
On Thu, May 23, 2019 at 06:06:43PM +0200, Moritz Barsnick wrote: > According to RFC1341, the multipart boundary indicated by the > Content-Type header must be prepended by CRLF + "--", and followed > by CRLF. In the case of strict MIME header boundary handling, the > "--" was forgotten to add. >

[FFmpeg-devel] [PATCH 2/2] avformat/mpjpegdec: fix strict boundary search string

2019-05-23 Thread Moritz Barsnick
According to RFC1341, the multipart boundary indicated by the Content-Type header must be prepended by CRLF + "--", and followed by CRLF. In the case of strict MIME header boundary handling, the "--" was forgotten to add. Fixes trac #7921. Signed-off-by: Moritz Barsnick ---