[FFmpeg-devel] [PATCH] lavf/http.c: Don't write chunked-transfer-trailer to listening server context.

[Michael Niedermayer]

On Tue, Apr 17, 2018 at 11:45:38PM +0200, Stephan Holljes wrote:
> On Thu, Apr 12, 2018 at 2:07 AM, Stephan Holljes
> <klaxa1337 at googlemail.com> wrote:
> > ---
> > I found this bug while working on my qualification task. When closing
> > a listening HTTP AVIOContext http_shutdown() tries to write the
> > chunked-transfer-encoding trailer to the listening Context.
> >
> >  libavformat/http.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/libavformat/http.c b/libavformat/http.c
> > index 983034f083..248fa085bf 100644
> > --- a/libavformat/http.c
> > +++ b/libavformat/http.c
> > @@ -1623,7 +1623,7 @@ static int http_shutdown(URLContext *h, int flags)
> >      HTTPContext *s = h->priv_data;
> >
> >      /* signal end of chunked encoding if used */
> > -    if (((flags & AVIO_FLAG_WRITE) && s->chunked_post) ||
> > +    if (((flags & AVIO_FLAG_WRITE) && s->chunked_post && !s->listen) ||
> >          ((flags & AVIO_FLAG_READ) && s->chunked_post && s->listen)) {
> >          ret = ffurl_write(s->hd, footer, sizeof(footer) - 1);
> >          ret = ret > 0 ? 0 : ret;
> > --
> > 2.16.2
> >
> 
> Ping

i think this is ok but we should find a maintainer for http

anyone who wants to help and knows http (either the protocol or our 
implementation)?

Thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If you think the mosad wants you dead since a long time then you are either
wrong or dead since a long time.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180419/7690d69b/attachment.sig>