Merge commit '5ae178539b91d25710b7bb322d156c31aea9f8bf'
authorMichael Niedermayer <michael@niedermayer.cc>
Thu, 30 Jul 2015 14:23:09 +0000 (16:23 +0200)
committerMichael Niedermayer <michael@niedermayer.cc>
Thu, 30 Jul 2015 14:24:03 +0000 (16:24 +0200)
* commit '5ae178539b91d25710b7bb322d156c31aea9f8bf':
  http: Add the trailing endlines if they are missing

Merged-by: Michael Niedermayer <michael@niedermayer.cc>
1  2 
libavformat/http.c

diff --combined libavformat/http.c
index 676bfd5c5a869c8f16b968f1af44fb974fd4fbd1,b3e9771757358f822b18ff00da899aca5f9bb231..684058147aec9c0b60ecf84a6394630612d8dcd0
@@@ -1,21 -1,21 +1,21 @@@
  /*
 - * HTTP protocol for avconv client
 + * HTTP protocol for ffmpeg client
   * Copyright (c) 2000, 2001 Fabrice Bellard
   *
 - * This file is part of Libav.
 + * This file is part of FFmpeg.
   *
 - * Libav is free software; you can redistribute it and/or
 + * FFmpeg is free software; you can redistribute it and/or
   * modify it under the terms of the GNU Lesser General Public
   * License as published by the Free Software Foundation; either
   * version 2.1 of the License, or (at your option) any later version.
   *
 - * Libav is distributed in the hope that it will be useful,
 + * FFmpeg is distributed in the hope that it will be useful,
   * but WITHOUT ANY WARRANTY; without even the implied warranty of
   * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
   * Lesser General Public License for more details.
   *
   * You should have received a copy of the GNU Lesser General Public
 - * License along with Libav; if not, write to the Free Software
 + * License along with FFmpeg; if not, write to the Free Software
   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
   */
  
@@@ -36,7 -36,7 +36,7 @@@
  #include "os_support.h"
  #include "url.h"
  
 -/* XXX: POST protocol is not completely implemented because avconv uses
 +/* XXX: POST protocol is not completely implemented because ffmpeg uses
   * only a subset of it. */
  
  /* The IO buffer size is unrelated to the max URL size in itself, but needs
@@@ -64,7 -64,6 +64,7 @@@ typedef struct HTTPContext 
      /* Set if the server correctly handles Connection: close and will close
       * the connection after feeding us the content. */
      int willclose;
 +    int seekable;           /**< Control seekability, 0 = disable, 1 = enable, -1 = probe. */
      int chunked_post;
      /* A flag which indicates if the end of chunked encoding has been sent. */
      int end_chunked_post;
      int multiple_requests;
      uint8_t *post_data;
      int post_datalen;
 +    int is_akamai;
 +    int is_mediagateway;
 +    char *cookies;          ///< holds newline (\n) delimited Set-Cookie header field values (without the "Set-Cookie: " field name)
 +    /* A dictionary containing cookies keyed by cookie name */
 +    AVDictionary *cookie_dict;
      int icy;
      /* how much data was read since the last ICY metadata packet */
      int icy_data_read;
@@@ -95,8 -89,6 +95,8 @@@
      AVDictionary *chained_options;
      int send_expect_100;
      char *method;
 +    int reconnect;
 +    int listen;
  } HTTPContext;
  
  #define OFFSET(x) offsetof(HTTPContext, x)
  #define DEFAULT_USER_AGENT "Lavf/" AV_STRINGIFY(LIBAVFORMAT_VERSION)
  
  static const AVOption options[] = {
 +    { "seekable", "control seekability of connection", OFFSET(seekable), AV_OPT_TYPE_INT, { .i64 = -1 }, -1, 1, D },
      { "chunked_post", "use chunked transfer-encoding for posts", OFFSET(chunked_post), AV_OPT_TYPE_INT, { .i64 = 1 }, 0, 1, E },
      { "headers", "set custom HTTP headers, can override built in default headers", OFFSET(headers), AV_OPT_TYPE_STRING, { 0 }, 0, 0, D | E },
      { "content_type", "set a specific content type for the POST messages", OFFSET(content_type), AV_OPT_TYPE_STRING, { 0 }, 0, 0, D | E },
      { "user_agent", "override User-Agent header", OFFSET(user_agent), AV_OPT_TYPE_STRING, { .str = DEFAULT_USER_AGENT }, 0, 0, D },
 -    { "user-agent", "override User-Agent header, for compatibility with ffmpeg", OFFSET(user_agent), AV_OPT_TYPE_STRING, { .str = DEFAULT_USER_AGENT }, 0, 0, D },
 +    { "user-agent", "override User-Agent header", OFFSET(user_agent), AV_OPT_TYPE_STRING, { .str = DEFAULT_USER_AGENT }, 0, 0, D },
      { "multiple_requests", "use persistent connections", OFFSET(multiple_requests), AV_OPT_TYPE_INT, { .i64 = 0 }, 0, 1, D | E },
      { "post_data", "set custom HTTP post data", OFFSET(post_data), AV_OPT_TYPE_BINARY, .flags = D | E },
      { "mime_type", "export the MIME type", OFFSET(mime_type), AV_OPT_TYPE_STRING, { 0 }, 0, 0, AV_OPT_FLAG_EXPORT | AV_OPT_FLAG_READONLY },
 +    { "cookies", "set cookies to be sent in applicable future requests, use newline delimited Set-Cookie HTTP field value syntax", OFFSET(cookies), AV_OPT_TYPE_STRING, { 0 }, 0, 0, D },
      { "icy", "request ICY metadata", OFFSET(icy), AV_OPT_TYPE_INT, { .i64 = 1 }, 0, 1, D },
      { "icy_metadata_headers", "return ICY metadata headers", OFFSET(icy_metadata_headers), AV_OPT_TYPE_STRING, { 0 }, 0, 0, AV_OPT_FLAG_EXPORT },
      { "icy_metadata_packet", "return current ICY metadata packet", OFFSET(icy_metadata_packet), AV_OPT_TYPE_STRING, { 0 }, 0, 0, AV_OPT_FLAG_EXPORT },
      { "location", "The actual location of the data received", OFFSET(location), AV_OPT_TYPE_STRING, { 0 }, 0, 0, D | E },
      { "offset", "initial byte offset", OFFSET(off), AV_OPT_TYPE_INT64, { .i64 = 0 }, 0, INT64_MAX, D },
      { "end_offset", "try to limit the request to bytes preceding this offset", OFFSET(end_off), AV_OPT_TYPE_INT64, { .i64 = 0 }, 0, INT64_MAX, D },
 -    { "method", "Override the HTTP method", OFFSET(method), AV_OPT_TYPE_STRING, { .str = NULL }, 0, 0, E },
 +    { "method", "Override the HTTP method or set the expected HTTP method from a client", OFFSET(method), AV_OPT_TYPE_STRING, { .str = NULL }, 0, 0, D | E },
 +    { "reconnect", "auto reconnect after disconnect before EOF", OFFSET(reconnect), AV_OPT_TYPE_INT, { .i64 = 0 }, 0, 1, D },
 +    { "listen", "listen on HTTP", OFFSET(listen), AV_OPT_TYPE_INT, { .i64 = 0 }, 0, 1, D | E },
      { NULL }
  };
  
  static int http_connect(URLContext *h, const char *path, const char *local_path,
                          const char *hoststr, const char *auth,
                          const char *proxyauth, int *new_location);
 +static int http_read_header(URLContext *h, int *new_location);
  
  void ff_http_init_auth_state(URLContext *dest, const URLContext *src)
  {
@@@ -214,8 -201,6 +214,8 @@@ static int http_open_cnx(URLContext *h
      HTTPContext *s = h->priv_data;
      int location_changed, attempts = 0, redirects = 0;
  redo:
 +    av_dict_copy(options, s->chained_options, 0);
 +
      cur_auth_type       = s->auth_state.auth_type;
      cur_proxy_auth_type = s->auth_state.auth_type;
  
      if (s->http_code == 401) {
          if ((cur_auth_type == HTTP_AUTH_NONE || s->auth_state.stale) &&
              s->auth_state.auth_type != HTTP_AUTH_NONE && attempts < 4) {
 -            ffurl_close(s->hd);
 -            s->hd = NULL;
 +            ffurl_closep(&s->hd);
              goto redo;
          } else
              goto fail;
      if (s->http_code == 407) {
          if ((cur_proxy_auth_type == HTTP_AUTH_NONE || s->proxy_auth_state.stale) &&
              s->proxy_auth_state.auth_type != HTTP_AUTH_NONE && attempts < 4) {
 -            ffurl_close(s->hd);
 -            s->hd = NULL;
 +            ffurl_closep(&s->hd);
              goto redo;
          } else
              goto fail;
           s->http_code == 303 || s->http_code == 307) &&
          location_changed == 1) {
          /* url moved, get next */
 -        ffurl_close(s->hd);
 -        s->hd = NULL;
 +        ffurl_closep(&s->hd);
          if (redirects++ >= MAX_REDIRECTS)
              return AVERROR(EIO);
          /* Restart the authentication process with the new target, which
  
  fail:
      if (s->hd)
 -        ffurl_close(s->hd);
 -    s->hd = NULL;
 -    return AVERROR(EIO);
 +        ffurl_closep(&s->hd);
 +    if (location_changed < 0)
 +        return location_changed;
 +    return ff_http_averror(s->http_code, AVERROR(EIO));
  }
  
  int ff_http_do_new_request(URLContext *h, const char *uri)
      if (!s->location)
          return AVERROR(ENOMEM);
  
 -    av_dict_copy(&options, s->chained_options, 0);
      ret = http_open_cnx(h, &options);
      av_dict_free(&options);
      return ret;
  }
  
 +int ff_http_averror(int status_code, int default_averror)
 +{
 +    switch (status_code) {
 +        case 400: return AVERROR_HTTP_BAD_REQUEST;
 +        case 401: return AVERROR_HTTP_UNAUTHORIZED;
 +        case 403: return AVERROR_HTTP_FORBIDDEN;
 +        case 404: return AVERROR_HTTP_NOT_FOUND;
 +        default: break;
 +    }
 +    if (status_code >= 400 && status_code <= 499)
 +        return AVERROR_HTTP_OTHER_4XX;
 +    else if (status_code >= 500)
 +        return AVERROR_HTTP_SERVER_ERROR;
 +    else
 +        return default_averror;
 +}
 +
 +static void handle_http_errors(URLContext *h, int error)
 +{
 +    static const char bad_request[] = "HTTP/1.1 400 Bad Request\r\nContent-Type: text/plain\r\n\r\n400 Bad Request\r\n";
 +    static const char internal_server_error[] = "HTTP/1.1 500 Internal server error\r\nContent-Type: text/plain\r\n\r\n500 Internal server error\r\n";
 +    HTTPContext *s = h->priv_data;
 +    if (h->is_connected) {
 +        switch(error) {
 +            case AVERROR_HTTP_BAD_REQUEST:
 +                ffurl_write(s->hd, bad_request, strlen(bad_request));
 +                break;
 +            default:
 +                av_log(h, AV_LOG_ERROR, "Unhandled HTTP error.\n");
 +                ffurl_write(s->hd, internal_server_error, strlen(internal_server_error));
 +        }
 +    }
 +}
 +
 +static int http_listen(URLContext *h, const char *uri, int flags,
 +                       AVDictionary **options) {
 +    HTTPContext *s = h->priv_data;
 +    int ret;
 +    static const char header[] = "HTTP/1.1 200 OK\r\nContent-Type: application/octet-stream\r\nTransfer-Encoding: chunked\r\n\r\n";
 +    char hostname[1024], proto[10];
 +    char lower_url[100];
 +    const char *lower_proto = "tcp";
 +    int port, new_location;
 +    s->chunked_post = 1;
 +    av_url_split(proto, sizeof(proto), NULL, 0, hostname, sizeof(hostname), &port,
 +                 NULL, 0, uri);
 +    if (!strcmp(proto, "https"))
 +        lower_proto = "tls";
 +    ff_url_join(lower_url, sizeof(lower_url), lower_proto, NULL, hostname, port,
 +                NULL);
 +    av_dict_set(options, "listen", "1", 0);
 +    if ((ret = ffurl_open(&s->hd, lower_url, AVIO_FLAG_READ_WRITE,
 +                          &h->interrupt_callback, options)) < 0)
 +        goto fail;
 +    if ((ret = http_read_header(h, &new_location)) < 0)
 +         goto fail;
 +    if ((ret = ffurl_write(s->hd, header, strlen(header))) < 0)
 +         goto fail;
 +    return 0;
 +
 +fail:
 +    handle_http_errors(h, ret);
 +    av_dict_free(&s->chained_options);
 +    return ret;
 +}
 +
  static int http_open(URLContext *h, const char *uri, int flags,
                       AVDictionary **options)
  {
      HTTPContext *s = h->priv_data;
      int ret;
  
 -    h->is_streamed = 1;
 +    if( s->seekable == 1 )
 +        h->is_streamed = 0;
 +    else
 +        h->is_streamed = 1;
  
      s->filesize = -1;
      s->location = av_strdup(uri);
  
      if (s->headers) {
          int len = strlen(s->headers);
-         if (len < 2 || strcmp("\r\n", s->headers + len - 2))
+         if (len < 2 || strcmp("\r\n", s->headers + len - 2)) {
              av_log(h, AV_LOG_WARNING,
                     "No trailing CRLF found in HTTP header.\n");
+             ret = av_reallocp(&s->headers, len + 3);
+             if (ret < 0)
+                 return ret;
+             s->headers[len]     = '\r';
+             s->headers[len + 1] = '\n';
+             s->headers[len + 2] = '\0';
+         }
      }
  
 +    if (s->listen) {
 +        return http_listen(h, uri, flags, options);
 +    }
      ret = http_open_cnx(h, options);
      if (ret < 0)
          av_dict_free(&s->chained_options);
@@@ -433,7 -356,7 +440,7 @@@ static int check_http_code(URLContext *
          (http_code != 407 || s->proxy_auth_state.auth_type != HTTP_AUTH_NONE)) {
          end += strspn(end, SPACE_CHARS);
          av_log(h, AV_LOG_WARNING, "HTTP error %d %s\n", http_code, end);
 -        return AVERROR(EIO);
 +        return ff_http_averror(http_code, AVERROR(EIO));
      }
      return 0;
  }
@@@ -463,8 -386,7 +470,8 @@@ static void parse_content_range(URLCont
          if ((slash = strchr(p, '/')) && strlen(slash) > 0)
              s->filesize = strtoll(slash + 1, NULL, 10);
      }
 -    h->is_streamed = 0; /* we _can_ in fact seek */
 +    if (s->seekable == -1 && (!s->is_akamai || s->filesize != 2147483647))
 +        h->is_streamed = 0; /* we _can_ in fact seek */
  }
  
  static int parse_content_encoding(URLContext *h, const char *p)
          // the header at all if this is the case).
      } else {
          av_log(h, AV_LOG_WARNING, "Unknown content coding: %s\n", p);
 -        return AVERROR(ENOSYS);
      }
      return 0;
  }
@@@ -523,49 -446,11 +530,49 @@@ static int parse_icy(HTTPContext *s, co
      return 0;
  }
  
 +static int parse_cookie(HTTPContext *s, const char *p, AVDictionary **cookies)
 +{
 +    char *eql, *name;
 +
 +    // duplicate the cookie name (dict will dupe the value)
 +    if (!(eql = strchr(p, '='))) return AVERROR(EINVAL);
 +    if (!(name = av_strndup(p, eql - p))) return AVERROR(ENOMEM);
 +
 +    // add the cookie to the dictionary
 +    av_dict_set(cookies, name, eql, AV_DICT_DONT_STRDUP_KEY);
 +
 +    return 0;
 +}
 +
 +static int cookie_string(AVDictionary *dict, char **cookies)
 +{
 +    AVDictionaryEntry *e = NULL;
 +    int len = 1;
 +
 +    // determine how much memory is needed for the cookies string
 +    while (e = av_dict_get(dict, "", e, AV_DICT_IGNORE_SUFFIX))
 +        len += strlen(e->key) + strlen(e->value) + 1;
 +
 +    // reallocate the cookies
 +    e = NULL;
 +    if (*cookies) av_free(*cookies);
 +    *cookies = av_malloc(len);
 +    if (!*cookies) return AVERROR(ENOMEM);
 +    *cookies[0] = '\0';
 +
 +    // write out the cookies
 +    while (e = av_dict_get(dict, "", e, AV_DICT_IGNORE_SUFFIX))
 +        av_strlcatf(*cookies, len, "%s%s\n", e->key, e->value);
 +
 +    return 0;
 +}
 +
  static int process_line(URLContext *h, char *line, int line_count,
                          int *new_location)
  {
      HTTPContext *s = h->priv_data;
 -    char *tag, *p, *end;
 +    const char *auto_method =  h->flags & AVIO_FLAG_READ ? "POST" : "GET";
 +    char *tag, *p, *end, *method, *resource, *version;
      int ret;
  
      /* end of header */
  
      p = line;
      if (line_count == 0) {
 -        while (!av_isspace(*p) && *p != '\0')
 -            p++;
 -        while (av_isspace(*p))
 -            p++;
 -        s->http_code = strtol(p, &end, 10);
 +        if (s->listen) {
 +            // HTTP method
 +            method = p;
 +            while (!av_isspace(*p))
 +                p++;
 +            *(p++) = '\0';
 +            av_log(h, AV_LOG_TRACE, "Received method: %s\n", method);
 +            if (s->method) {
 +                if (av_strcasecmp(s->method, method)) {
 +                    av_log(h, AV_LOG_ERROR, "Received and expected HTTP method do not match. (%s expected, %s received)\n",
 +                           s->method, method);
 +                    return ff_http_averror(400, AVERROR(EIO));
 +                }
 +            } else {
 +                // use autodetected HTTP method to expect
 +                av_log(h, AV_LOG_TRACE, "Autodetected %s HTTP method\n", auto_method);
 +                if (av_strcasecmp(auto_method, method)) {
 +                    av_log(h, AV_LOG_ERROR, "Received and autodetected HTTP method did not match "
 +                           "(%s autodetected %s received)\n", auto_method, method);
 +                    return ff_http_averror(400, AVERROR(EIO));
 +                }
 +            }
  
 -        av_log(NULL, AV_LOG_TRACE, "http_code=%d\n", s->http_code);
 +            // HTTP resource
 +            while (av_isspace(*p))
 +                p++;
 +            resource = p;
 +            while (!av_isspace(*p))
 +                p++;
 +            *(p++) = '\0';
 +            av_log(h, AV_LOG_TRACE, "Requested resource: %s\n", resource);
 +
 +            // HTTP version
 +            while (av_isspace(*p))
 +                p++;
 +            version = p;
 +            while (!av_isspace(*p))
 +                p++;
 +            *p = '\0';
 +            if (av_strncasecmp(version, "HTTP/", 5)) {
 +                av_log(h, AV_LOG_ERROR, "Malformed HTTP version string.\n");
 +                return ff_http_averror(400, AVERROR(EIO));
 +            }
 +            av_log(h, AV_LOG_TRACE, "HTTP version string: %s\n", version);
 +        } else {
 +            while (!av_isspace(*p) && *p != '\0')
 +                p++;
 +            while (av_isspace(*p))
 +                p++;
 +            s->http_code = strtol(p, &end, 10);
  
 -        if ((ret = check_http_code(h, s->http_code, end)) < 0)
 -            return ret;
 +            av_log(h, AV_LOG_TRACE, "http_code=%d\n", s->http_code);
 +
 +            if ((ret = check_http_code(h, s->http_code, end)) < 0)
 +                return ret;
 +        }
      } else {
          while (*p != '\0' && *p != ':')
              p++;
          } else if (!av_strcasecmp(tag, "Content-Range")) {
              parse_content_range(h, p);
          } else if (!av_strcasecmp(tag, "Accept-Ranges") &&
 -                   !strncmp(p, "bytes", 5)) {
 +                   !strncmp(p, "bytes", 5) &&
 +                   s->seekable == -1) {
              h->is_streamed = 0;
          } else if (!av_strcasecmp(tag, "Transfer-Encoding") &&
                     !av_strncasecmp(p, "chunked", 7)) {
          } else if (!av_strcasecmp(tag, "Connection")) {
              if (!strcmp(p, "close"))
                  s->willclose = 1;
 +        } else if (!av_strcasecmp(tag, "Server")) {
 +            if (!av_strcasecmp(p, "AkamaiGHost")) {
 +                s->is_akamai = 1;
 +            } else if (!av_strncasecmp(p, "MediaGateway", 12)) {
 +                s->is_mediagateway = 1;
 +            }
          } else if (!av_strcasecmp(tag, "Content-Type")) {
              av_free(s->mime_type);
              s->mime_type = av_strdup(p);
 +        } else if (!av_strcasecmp(tag, "Set-Cookie")) {
 +            if (parse_cookie(s, p, &s->cookie_dict))
 +                av_log(h, AV_LOG_WARNING, "Unable to parse '%s'\n", p);
          } else if (!av_strcasecmp(tag, "Icy-MetaInt")) {
              s->icy_metaint = strtoll(p, NULL, 10);
          } else if (!av_strncasecmp(tag, "Icy-", 4)) {
      return 1;
  }
  
 +/**
 + * Create a string containing cookie values for use as a HTTP cookie header
 + * field value for a particular path and domain from the cookie values stored in
 + * the HTTP protocol context. The cookie string is stored in *cookies.
 + *
 + * @return a negative value if an error condition occurred, 0 otherwise
 + */
 +static int get_cookies(HTTPContext *s, char **cookies, const char *path,
 +                       const char *domain)
 +{
 +    // cookie strings will look like Set-Cookie header field values.  Multiple
 +    // Set-Cookie fields will result in multiple values delimited by a newline
 +    int ret = 0;
 +    char *next, *cookie, *set_cookies = av_strdup(s->cookies), *cset_cookies = set_cookies;
 +
 +    if (!set_cookies) return AVERROR(EINVAL);
 +
 +    // destroy any cookies in the dictionary.
 +    av_dict_free(&s->cookie_dict);
 +
 +    *cookies = NULL;
 +    while ((cookie = av_strtok(set_cookies, "\n", &next))) {
 +        int domain_offset = 0;
 +        char *param, *next_param, *cdomain = NULL, *cpath = NULL, *cvalue = NULL;
 +        set_cookies = NULL;
 +
 +        // store the cookie in a dict in case it is updated in the response
 +        if (parse_cookie(s, cookie, &s->cookie_dict))
 +            av_log(s, AV_LOG_WARNING, "Unable to parse '%s'\n", cookie);
 +
 +        while ((param = av_strtok(cookie, "; ", &next_param))) {
 +            if (cookie) {
 +                // first key-value pair is the actual cookie value
 +                cvalue = av_strdup(param);
 +                cookie = NULL;
 +            } else if (!av_strncasecmp("path=",   param, 5)) {
 +                av_free(cpath);
 +                cpath = av_strdup(&param[5]);
 +            } else if (!av_strncasecmp("domain=", param, 7)) {
 +                // if the cookie specifies a sub-domain, skip the leading dot thereby
 +                // supporting URLs that point to sub-domains and the master domain
 +                int leading_dot = (param[7] == '.');
 +                av_free(cdomain);
 +                cdomain = av_strdup(&param[7+leading_dot]);
 +            } else {
 +                // ignore unknown attributes
 +            }
 +        }
 +        if (!cdomain)
 +            cdomain = av_strdup(domain);
 +
 +        // ensure all of the necessary values are valid
 +        if (!cdomain || !cpath || !cvalue) {
 +            av_log(s, AV_LOG_WARNING,
 +                   "Invalid cookie found, no value, path or domain specified\n");
 +            goto done_cookie;
 +        }
 +
 +        // check if the request path matches the cookie path
 +        if (av_strncasecmp(path, cpath, strlen(cpath)))
 +            goto done_cookie;
 +
 +        // the domain should be at least the size of our cookie domain
 +        domain_offset = strlen(domain) - strlen(cdomain);
 +        if (domain_offset < 0)
 +            goto done_cookie;
 +
 +        // match the cookie domain
 +        if (av_strcasecmp(&domain[domain_offset], cdomain))
 +            goto done_cookie;
 +
 +        // cookie parameters match, so copy the value
 +        if (!*cookies) {
 +            if (!(*cookies = av_strdup(cvalue))) {
 +                ret = AVERROR(ENOMEM);
 +                goto done_cookie;
 +            }
 +        } else {
 +            char *tmp = *cookies;
 +            size_t str_size = strlen(cvalue) + strlen(*cookies) + 3;
 +            if (!(*cookies = av_malloc(str_size))) {
 +                ret = AVERROR(ENOMEM);
 +                goto done_cookie;
 +            }
 +            snprintf(*cookies, str_size, "%s; %s", tmp, cvalue);
 +            av_free(tmp);
 +        }
 +
 +        done_cookie:
 +        av_freep(&cdomain);
 +        av_freep(&cpath);
 +        av_freep(&cvalue);
 +        if (ret < 0) {
 +            if (*cookies) av_freep(cookies);
 +            av_free(cset_cookies);
 +            return ret;
 +        }
 +    }
 +
 +    av_free(cset_cookies);
 +
 +    return 0;
 +}
 +
  static inline int has_header(const char *str, const char *header)
  {
      /* header + 2 to skip over CRLF prefix. (make sure you have one!) */
@@@ -817,7 -542,7 +824,7 @@@ static int http_read_header(URLContext 
          if ((err = http_get_line(s, line, sizeof(line))) < 0)
              return err;
  
 -        av_log(NULL, AV_LOG_TRACE, "header='%s'\n", line);
 +        av_log(h, AV_LOG_TRACE, "header='%s'\n", line);
  
          err = process_line(h, line, s->line_count, new_location);
          if (err < 0)
          s->line_count++;
      }
  
 +    if (s->seekable == -1 && s->is_mediagateway && s->filesize == 2000000000)
 +        h->is_streamed = 1; /* we can in fact _not_ seek */
 +
 +    // add any new cookies into the existing cookie string
 +    cookie_string(s->cookie_dict, &s->cookies);
 +    av_dict_free(&s->cookie_dict);
 +
      return err;
  }
  
@@@ -891,7 -609,7 +898,7 @@@ static int http_connect(URLContext *h, 
      // Note: we send this on purpose even when s->off is 0 when we're probing,
      // since it allows us to detect more reliably if a (non-conforming)
      // server supports seeking by analysing the reply headers.
 -    if (!has_header(s->headers, "\r\nRange: ") && !post) {
 +    if (!has_header(s->headers, "\r\nRange: ") && !post && (s->off > 0 || s->end_off || s->seekable == -1)) {
          len += av_strlcatf(headers + len, sizeof(headers) - len,
                             "Range: bytes=%"PRId64"-", s->off);
          if (s->end_off)
      if (!has_header(s->headers, "\r\nContent-Type: ") && s->content_type)
          len += av_strlcatf(headers + len, sizeof(headers) - len,
                             "Content-Type: %s\r\n", s->content_type);
 +    if (!has_header(s->headers, "\r\nCookie: ") && s->cookies) {
 +        char *cookies = NULL;
 +        if (!get_cookies(s, &cookies, path, hoststr) && cookies) {
 +            len += av_strlcatf(headers + len, sizeof(headers) - len,
 +                               "Cookie: %s\r\n", cookies);
 +            av_free(cookies);
 +        }
 +    }
      if (!has_header(s->headers, "\r\nIcy-MetaData: ") && s->icy)
          len += av_strlcatf(headers + len, sizeof(headers) - len,
                             "Icy-MetaData: %d\r\n", 1);
               authstr ? authstr : "",
               proxyauthstr ? "Proxy-" : "", proxyauthstr ? proxyauthstr : "");
  
 -    av_freep(&authstr);
 -    av_freep(&proxyauthstr);
 +    av_log(h, AV_LOG_DEBUG, "request: %s\n", s->buffer);
 +
      if ((err = ffurl_write(s->hd, s->buffer, strlen(s->buffer))) < 0)
 -        return err;
 +        goto done;
  
      if (s->post_data)
          if ((err = ffurl_write(s->hd, s->post_data, s->post_datalen)) < 0)
 -            return err;
 +            goto done;
  
      /* init input buffer */
      s->buf_ptr          = s->buffer;
           * we've still to send the POST data, but the code calling this
           * function will check http_code after we return. */
          s->http_code = 200;
 -        return 0;
 +        err = 0;
 +        goto done;
      }
  
      /* wait for header */
      err = http_read_header(h, new_location);
      if (err < 0)
 -        return err;
 +        goto done;
  
 -    return (off == s->off) ? 0 : -1;
 +    if (*new_location)
 +        s->off = off;
 +
 +    err = (off == s->off) ? 0 : -1;
 +done:
 +    av_freep(&authstr);
 +    av_freep(&proxyauthstr);
 +    return err;
  }
  
  static int http_buf_read(URLContext *h, uint8_t *buf, int size)
              s->filesize >= 0 && s->off >= s->filesize)
              return AVERROR_EOF;
          len = ffurl_read(s->hd, buf, size);
 +        if (!len && (!s->willclose || s->chunksize < 0) &&
 +            s->filesize >= 0 && s->off < s->filesize) {
 +            av_log(h, AV_LOG_ERROR,
 +                   "Stream ends prematurely at %"PRId64", should be %"PRId64"\n",
 +                   s->off, s->filesize
 +                  );
 +            return AVERROR(EIO);
 +        }
      }
      if (len > 0) {
          s->off += len;
@@@ -1062,12 -756,10 +1069,12 @@@ static int http_buf_read_compressed(URL
  }
  #endif /* CONFIG_ZLIB */
  
 +static int64_t http_seek_internal(URLContext *h, int64_t off, int whence, int force_reconnect);
 +
  static int http_read_stream(URLContext *h, uint8_t *buf, int size)
  {
      HTTPContext *s = h->priv_data;
 -    int err, new_location;
 +    int err, new_location, read_ret, seek_ret;
  
      if (!s->hd)
          return AVERROR_EOF;
          if (!s->chunksize) {
              char line[32];
  
 -            for (;;) {
                  do {
                      if ((err = http_get_line(s, line, sizeof(line))) < 0)
                          return err;
  
                  if (!s->chunksize)
                      return 0;
 -                break;
 -            }
          }
          size = FFMIN(size, s->chunksize);
      }
      if (s->compressed)
          return http_buf_read_compressed(h, buf, size);
  #endif /* CONFIG_ZLIB */
 -    return http_buf_read(h, buf, size);
 +    read_ret = http_buf_read(h, buf, size);
 +    if (read_ret < 0 && s->reconnect && !h->is_streamed && s->filesize > 0 && s->off < s->filesize) {
 +        av_log(h, AV_LOG_INFO, "Will reconnect at %"PRId64".\n", s->off);
 +        seek_ret = http_seek_internal(h, s->off, SEEK_SET, 1);
 +        if (seek_ret != s->off) {
 +            av_log(h, AV_LOG_ERROR, "Failed to reconnect at %"PRId64".\n", s->off);
 +            return read_ret;
 +        }
 +
 +        read_ret = http_buf_read(h, buf, size);
 +    }
 +
 +    return read_ret;
  }
  
 +// Like http_read_stream(), but no short reads.
 +// Assumes partial reads are an error.
  static int http_read_stream_all(URLContext *h, uint8_t *buf, int size)
  {
      int pos = 0;
@@@ -1243,8 -924,7 +1250,8 @@@ static int http_shutdown(URLContext *h
      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) ||
 +        ((flags & AVIO_FLAG_READ) && s->chunked_post && s->listen)) {
          ret = ffurl_write(s->hd, footer, sizeof(footer) - 1);
          ret = ret > 0 ? 0 : ret;
          s->end_chunked_post = 1;
@@@ -1268,12 -948,12 +1275,12 @@@ static int http_close(URLContext *h
          ret = http_shutdown(h, h->flags);
  
      if (s->hd)
 -        ffurl_close(s->hd);
 +        ffurl_closep(&s->hd);
      av_dict_free(&s->chained_options);
      return ret;
  }
  
 -static int64_t http_seek(URLContext *h, int64_t off, int whence)
 +static int64_t http_seek_internal(URLContext *h, int64_t off, int whence, int force_reconnect)
  {
      HTTPContext *s = h->priv_data;
      URLContext *old_hd = s->hd;
  
      if (whence == AVSEEK_SIZE)
          return s->filesize;
 -    else if ((whence == SEEK_CUR && off == 0) ||
 -             (whence == SEEK_SET && off == s->off))
 +    else if (!force_reconnect &&
 +             ((whence == SEEK_CUR && off == 0) ||
 +              (whence == SEEK_SET && off == s->off)))
          return s->off;
      else if ((s->filesize == -1 && whence == SEEK_END) || h->is_streamed)
          return AVERROR(ENOSYS);
  
 -    /* we save the old context in case the seek fails */
 -    old_buf_size = s->buf_end - s->buf_ptr;
 -    memcpy(old_buf, s->buf_ptr, old_buf_size);
 -    s->hd = NULL;
      if (whence == SEEK_CUR)
          off += s->off;
      else if (whence == SEEK_END)
          off += s->filesize;
 +    else if (whence != SEEK_SET)
 +        return AVERROR(EINVAL);
 +    if (off < 0)
 +        return AVERROR(EINVAL);
      s->off = off;
  
 +    /* we save the old context in case the seek fails */
 +    old_buf_size = s->buf_end - s->buf_ptr;
 +    memcpy(old_buf, s->buf_ptr, old_buf_size);
 +    s->hd = NULL;
 +
      /* if it fails, continue on old connection */
 -    av_dict_copy(&options, s->chained_options, 0);
      if ((ret = http_open_cnx(h, &options)) < 0) {
          av_dict_free(&options);
          memcpy(s->buffer, old_buf, old_buf_size);
      return off;
  }
  
 +static int64_t http_seek(URLContext *h, int64_t off, int whence)
 +{
 +    return http_seek_internal(h, off, whence, 0);
 +}
 +
  static int http_get_file_handle(URLContext *h)
  {
      HTTPContext *s = h->priv_data;
@@@ -1381,7 -1051,7 +1388,7 @@@ static int http_proxy_close(URLContext 
  {
      HTTPContext *s = h->priv_data;
      if (s->hd)
 -        ffurl_close(s->hd);
 +        ffurl_closep(&s->hd);
      return 0;
  }
  
@@@ -1396,10 -1066,7 +1403,10 @@@ static int http_proxy_open(URLContext *
      char *authstr;
      int new_loc;
  
 -    h->is_streamed = 1;
 +    if( s->seekable == 1 )
 +        h->is_streamed = 0;
 +    else
 +        h->is_streamed = 1;
  
      av_url_split(NULL, 0, auth, sizeof(auth), hostname, sizeof(hostname), &port,
                   pathbuf, sizeof(pathbuf), uri);
@@@ -1455,13 -1122,14 +1462,13 @@@ redo
      if (s->http_code == 407 &&
          (cur_auth_type == HTTP_AUTH_NONE || s->proxy_auth_state.stale) &&
          s->proxy_auth_state.auth_type != HTTP_AUTH_NONE && attempts < 2) {
 -        ffurl_close(s->hd);
 -        s->hd = NULL;
 +        ffurl_closep(&s->hd);
          goto redo;
      }
  
      if (s->http_code < 400)
          return 0;
 -    ret = AVERROR(EIO);
 +    ret = ff_http_averror(s->http_code, AVERROR(EIO));
  
  fail:
      http_proxy_close(h);