[FFmpeg-devel] Proposed patches to mpjpeg demux

Alex Agranovsky alex at sighthound.com
Thu Sep 10 18:04:53 CEST 2015

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: ffmpeg3.patch
Type: application/octet-stream
Size: 1967 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150910/1e5c5235/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffmpeg4.patch
Type: application/octet-stream
Size: 603 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150910/1e5c5235/attachment-0001.obj>

More information about the ffmpeg-devel mailing list