On Fri, May 29, 2015 at 4:19 PM, wm4 <[email protected]> wrote: > On Fri, 29 May 2015 16:12:27 +0200 > Stephan Holljes <[email protected]> wrote: > >> Hi, >> >> On Fri, May 29, 2015 at 11:54 AM, Michael Niedermayer <[email protected]> >> wrote: >> > On Fri, May 29, 2015 at 05:05:08AM +0200, Stephan Holljes wrote: >> >> Signed-off-by: Stephan Holljes <[email protected]> >> >> --- >> >> libavformat/http.c | 8 ++++++++ >> >> 1 file changed, 8 insertions(+) >> >> >> >> diff --git a/libavformat/http.c b/libavformat/http.c >> >> index 4f6716a..3fad43d 100644 >> >> --- a/libavformat/http.c >> >> +++ b/libavformat/http.c >> >> @@ -556,6 +556,14 @@ static int process_line(URLContext *h, char *line, >> >> int line_count, >> >> >> >> p = line; >> >> if (line_count == 0) { >> >> + if (s->listen) { >> >> + while (!av_isspace(*p)) >> >> + p++; >> > >> >> + s->method = av_malloc(p - line); >> >> + if (!s->method) >> >> + return AVERROR(ENOMEM); >> >> + av_strlcpy(s->method, line, p - line + 1); >> > >> > the allocated size and the size passed to av_strlcpy() should match >> >> Attached patch fixes that. >> > > av_strndup?
That looks a lot more suited. Will change the code accordingly. > _______________________________________________ > ffmpeg-devel mailing list > [email protected] > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel _______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
