[FFmpeg-devel] [PATCH]Add one CRLF to http headers if necessary
Derek Buitenhuis
derek.buitenhuis at gmail.com
Thu Feb 26 15:31:39 CET 2015
On 2/26/2015 2:09 PM, Carl Eugen Hoyos wrote:
> + snprintf(header, len + 3, "%s\r\n", s->headers);
This does not necessarily work on windows. The C standard idctates that in
text mode, \n is translated to the system's native newline.
Use memcpy and 0x0D / 0X0A / 0x00.
This may also accidentally allow for headers to end with '\n\r\n',
wouldn't it?
- Derek
More information about the ffmpeg-devel
mailing list