[FFmpeg-devel] [PATCH 1/2] http: Improve handling of Content-Range with Transfer-Encoding:chunked
Aman Karmani
ffmpeg at tmm1.net
Thu Feb 3 03:26:52 EET 2022
On Wed, Feb 2, 2022 at 3:50 PM Vittorio Giovara <vittorio.giovara at gmail.com>
wrote:
> From: Justin Ruggles <justin.ruggles at gmail.com>
>
> When Transfer-Encoding:chunked is used, the client must ignore a
> Content-Length header, if present. However, it should not ignore a
> Content-Range header, which also includes the full size of the
> entity.
> ---
> libavformat/http.c | 8 +++++++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/libavformat/http.c b/libavformat/http.c
> index 8f39d11a88..c89f8a5517 100644
> --- a/libavformat/http.c
> +++ b/libavformat/http.c
> @@ -132,6 +132,7 @@ typedef struct HTTPContext {
> int64_t expires;
> char *new_location;
> AVDictionary *redirect_cache;
> + uint64_t filesize_from_content_range;
> } HTTPContext;
>
> #define OFFSET(x) offsetof(HTTPContext, x)
> @@ -839,7 +840,7 @@ static void parse_content_range(URLContext *h, const
> char *p)
> p += 6;
> s->off = strtoull(p, NULL, 10);
> if ((slash = strchr(p, '/')) && strlen(slash) > 0)
> - s->filesize = strtoull(slash + 1, NULL, 10);
> + s->filesize_from_content_range = strtoull(slash + 1, NULL,
> 10);
>
The size part of the range header is optional, and can be '*' as well.
See also
https://patchwork.ffmpeg.org/project/ffmpeg/patch/20211005233244.37582-1-ffmpeg@tmm1.net/
> }
> if (s->seekable == -1 && (!s->is_akamai || s->filesize != 2147483647))
> h->is_streamed = 0; /* we _can_ in fact seek */
> @@ -1341,6 +1342,7 @@ static int http_read_header(URLContext *h)
> av_freep(&s->new_location);
> s->expires = 0;
> s->chunksize = UINT64_MAX;
> + s->filesize_from_content_range = UINT64_MAX;
>
> for (;;) {
> if ((err = http_get_line(s, line, sizeof(line))) < 0)
> @@ -1356,6 +1358,10 @@ static int http_read_header(URLContext *h)
> s->line_count++;
> }
>
> + // filesize from Content-Range can always be used, even if using
> chunked Transfer-Encoding
> + if (s->filesize_from_content_range != UINT64_MAX)
> + s->filesize = s->filesize_from_content_range;
> +
> if (s->seekable == -1 && s->is_mediagateway && s->filesize ==
> 2000000000)
> h->is_streamed = 1; /* we can in fact _not_ seek */
>
> --
> 2.34.1
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
>
More information about the ffmpeg-devel
mailing list