[FFmpeg-devel] [PATCH][GSoC] lavf/http: Parse and set HTTP method when listening on HTTP(S)

[Stephan Holljes]

On Fri, May 29, 2015 at 4:19 PM, wm4 <nfxjfg at googlemail.com> wrote:
> On Fri, 29 May 2015 16:12:27 +0200
> Stephan Holljes <klaxa1337 at googlemail.com> wrote:
>
>> Hi,
>>
>> On Fri, May 29, 2015 at 11:54 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
>> > On Fri, May 29, 2015 at 05:05:08AM +0200, Stephan Holljes wrote:
>> >> Signed-off-by: Stephan Holljes <klaxa1337 at googlemail.com>
>> >> ---
>> >>  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
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel