[FFmpeg-devel] Proposed patches to mpjpeg demux

[Alex Agranovsky]

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



-- 
Alex Agranovsky
Sighthound, Inc
www.sighthound.com		

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
www.sighthound.com
-------------- 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>