[FFmpeg-devel] [PATCH] avformat/http: detect end of received on chunked persistenc

[Michael Niedermayer]

On Tue, Sep 22, 2015 at 05:02:38AM +0000, Schenk, Michael wrote:
> Dear friends,
> we had a problem with an HLS server which was using chunked http transfer over persistence http connection.
> Even if we have received all data (chunksize = 0) the read of a m3u8 playlist was blocked till the server tears down the TCP
> connection. Please find attached a patch for avformat/http.c which will mark end of received once all chunks
> are transferred completely. I sent this a second time because I did not see the first eMail of me.
> 
> Feedback for sure is warmly welcome.
> 
> Regards
> 
> Michael
> 

>  http.c |    5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 8b122aae4c2964d45df3527f4217d267221aee9c  0001-avformat-http-detect-end-of-received-on-chunked-pers.patch
> From 511c4f7b50c4abf21e482636379021f14451be0d Mon Sep 17 00:00:00 2001
> From: "Schenk, Michael" <michael.schenk at albistechnologies.com>
> Date: Mon, 21 Sep 2015 15:15:11 +0200
> Subject: [PATCH] avformat/http: detect end of received on chunked persistence
>  http connections
> 
> ---
>  libavformat/http.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/libavformat/http.c b/libavformat/http.c
> index 075dda7..5f48e6e 100644
> --- a/libavformat/http.c
> +++ b/libavformat/http.c
> @@ -1239,8 +1239,11 @@ static int http_read_stream(URLContext *h, uint8_t *buf, int size)
>                  av_log(NULL, AV_LOG_TRACE, "Chunked encoding data size: %"PRId64"'\n",
>                          s->chunksize);
>  
> -                if (!s->chunksize)
> +                if (!s->chunksize) {
> +                    s->chunksize = -1;

> +                    s->filesize = s->off;

is this related or a seperate unrelated improvment ?

also, is there a public server to test the chunksize bugfix ?

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150924/967b1b66/attachment.sig>