[FFmpeg-devel] Proposed patches to mpjpeg demux

Alex Agranovsky alex at sighthound.com
Thu Sep 10 23:23:09 CEST 2015

Quick update: format.c patch needs to be amended, to include a NULL check.

Alex Agranovsky
Sighthound, Inc

On September 10, 2015 at 12:04:55 PM, Alex Agranovsky (alex at sighthound.com) wrote:

Please look at the attached patches. They accomplish the following:

1) change to format.c: allows the demux to be matched by Content-Type, even if the incoming Content-Type header contains parameters, such as 'multipart/x-mixed-replace;boundary=ffserver’. Currently the comparison seems to occur verbatim, thus preventing the selection of mpjpegdec demux, even when running against MJPEG served by ffserver.
P.S. I’m not sure if parameters are ever needed to be included for this comparison, but considering we’re dealing with probing, seems unlikely

2) changes to mpjpegdec.c:
- allow the empty line after multipart MIME headers to be parsed without generating an error
- if get_line encounters EOF, don’t fail right away (exit the loop and check if the relevant headers had been processed instead)
- trim whitespace from both name and value extracted from a header, allowing for more flexibility for the incoming data

Alex Agranovsky
Sighthound, Inc
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffmpeg4.patch
Type: application/octet-stream
Size: 621 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150910/70cab147/attachment.obj>

More information about the ffmpeg-devel mailing list