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" <[email protected]>
> 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

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
[email protected]
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to