Merge branch 'sp/maint-clear-postfields' into maint
authorJunio C Hamano <gitster@pobox.com>
Wed, 4 May 2011 21:58:56 +0000 (14:58 -0700)
committerJunio C Hamano <gitster@pobox.com>
Wed, 4 May 2011 21:58:56 +0000 (14:58 -0700)
* sp/maint-clear-postfields:
http: clear POSTFIELDS when initializing a slot

1  2 
http.c
diff --combined http.c
index 9e767723ed1045445e91e79489cb8ec8bb861965,b642eac978025a1078f3b7f6233083f0d5dbddfa..b27bb57d627f19d725b1f24aa3f06d261e48f8e5
--- 1/http.c
--- 2/http.c
+++ b/http.c
@@@ -1,20 -1,12 +1,20 @@@
  #include "http.h"
  #include "pack.h"
  #include "sideband.h"
 +#include "run-command.h"
 +#include "url.h"
  
  int data_received;
  int active_requests;
  int http_is_verbose;
  size_t http_post_buffer = 16 * LARGE_PACKET_MAX;
  
 +#if LIBCURL_VERSION_NUM >= 0x070a06
 +#define LIBCURL_CAN_HANDLE_AUTH_ANY
 +#endif
 +
 +static int min_curl_sessions = 1;
 +static int curl_session_count;
  #ifdef USE_CURL_MULTI
  static int max_requests = -1;
  static CURLM *curlm;
@@@ -42,7 -34,6 +42,7 @@@ static long curl_low_speed_time = -1
  static int curl_ftp_no_epsv;
  static const char *curl_http_proxy;
  static char *user_name, *user_pass;
 +static const char *user_agent;
  
  #if LIBCURL_VERSION_NUM >= 0x071700
  /* Use CURLOPT_KEYPASSWD as is */
@@@ -161,14 -152,6 +161,14 @@@ static int http_options(const char *var
                        ssl_cert_password_required = 1;
                return 0;
        }
 +      if (!strcmp("http.minsessions", var)) {
 +              min_curl_sessions = git_config_int(var, value);
 +#ifndef USE_CURL_MULTI
 +              if (min_curl_sessions > 1)
 +                      min_curl_sessions = 1;
 +#endif
 +              return 0;
 +      }
  #ifdef USE_CURL_MULTI
        if (!strcmp("http.maxrequests", var)) {
                max_requests = git_config_int(var, value);
                return 0;
        }
  
 +      if (!strcmp("http.useragent", var))
 +              return git_config_string(&user_agent, var, value);
 +
        /* Fall back on the default ones */
        return git_default_config(var, value, cb);
  }
@@@ -210,7 -190,7 +210,7 @@@ static void init_curl_http_auth(CURL *r
        if (user_name) {
                struct strbuf up = STRBUF_INIT;
                if (!user_pass)
 -                      user_pass = xstrdup(getpass("Password: "));
 +                      user_pass = xstrdup(git_getpass("Password: "));
                strbuf_addf(&up, "%s:%s", user_name, user_pass);
                curl_easy_setopt(result, CURLOPT_USERPWD,
                                 strbuf_detach(&up, NULL));
@@@ -225,7 -205,7 +225,7 @@@ static int has_cert_password(void
                return 0;
        /* Only prompt the user once. */
        ssl_cert_password_required = -1;
 -      ssl_cert_password = getpass("Certificate Password: ");
 +      ssl_cert_password = git_getpass("Certificate Password: ");
        if (ssl_cert_password != NULL) {
                ssl_cert_password = xstrdup(ssl_cert_password);
                return 1;
@@@ -250,9 -230,6 +250,9 @@@ static CURL *get_curl_handle(void
  #if LIBCURL_VERSION_NUM >= 0x070907
        curl_easy_setopt(result, CURLOPT_NETRC, CURL_NETRC_OPTIONAL);
  #endif
 +#ifdef LIBCURL_CAN_HANDLE_AUTH_ANY
 +      curl_easy_setopt(result, CURLOPT_HTTPAUTH, CURLAUTH_ANY);
 +#endif
  
        init_curl_http_auth(result);
  
        }
  
        curl_easy_setopt(result, CURLOPT_FOLLOWLOCATION, 1);
 +#if LIBCURL_VERSION_NUM >= 0x071301
 +      curl_easy_setopt(result, CURLOPT_POSTREDIR, CURL_REDIR_POST_ALL);
 +#elif LIBCURL_VERSION_NUM >= 0x071101
 +      curl_easy_setopt(result, CURLOPT_POST301, 1);
 +#endif
  
        if (getenv("GIT_CURL_VERBOSE"))
                curl_easy_setopt(result, CURLOPT_VERBOSE, 1);
  
 -      curl_easy_setopt(result, CURLOPT_USERAGENT, GIT_USER_AGENT);
 +      curl_easy_setopt(result, CURLOPT_USERAGENT,
 +              user_agent ? user_agent : GIT_HTTP_USER_AGENT);
  
        if (curl_ftp_no_epsv)
                curl_easy_setopt(result, CURLOPT_FTP_USE_EPSV, 0);
  
  static void http_auth_init(const char *url)
  {
 -      char *at, *colon, *cp, *slash;
 +      char *at, *colon, *cp, *slash, *decoded;
        int len;
  
        cp = strstr(url, "://");
                user_name = xmalloc(len + 1);
                memcpy(user_name, cp, len);
                user_name[len] = '\0';
 +              decoded = url_decode(user_name);
 +              free(user_name);
 +              user_name = decoded;
                user_pass = NULL;
        } else {
                len = colon - cp;
                user_name = xmalloc(len + 1);
                memcpy(user_name, cp, len);
                user_name[len] = '\0';
 +              decoded = url_decode(user_name);
 +              free(user_name);
 +              user_name = decoded;
                len = at - (colon + 1);
                user_pass = xmalloc(len + 1);
                memcpy(user_pass, colon + 1, len);
                user_pass[len] = '\0';
 +              decoded = url_decode(user_pass);
 +              free(user_pass);
 +              user_pass = decoded;
        }
  }
  
@@@ -400,8 -362,6 +400,8 @@@ void http_init(struct remote *remote
  #endif
        set_from_env(&ssl_cainfo, "GIT_SSL_CAINFO");
  
 +      set_from_env(&user_agent, "GIT_HTTP_USER_AGENT");
 +
        low_speed_limit = getenv("GIT_HTTP_LOW_SPEED_LIMIT");
        if (low_speed_limit != NULL)
                curl_low_speed_limit = strtol(low_speed_limit, NULL, 10);
        if (curl_ssl_verify == -1)
                curl_ssl_verify = 1;
  
 +      curl_session_count = 0;
  #ifdef USE_CURL_MULTI
        if (max_requests < 1)
                max_requests = DEFAULT_MAX_REQUESTS;
@@@ -521,7 -480,6 +521,7 @@@ struct active_request_slot *get_active_
  #else
                slot->curl = curl_easy_duphandle(curl_default);
  #endif
 +              curl_session_count++;
        }
  
        active_requests++;
        curl_easy_setopt(slot->curl, CURLOPT_CUSTOMREQUEST, NULL);
        curl_easy_setopt(slot->curl, CURLOPT_READFUNCTION, NULL);
        curl_easy_setopt(slot->curl, CURLOPT_WRITEFUNCTION, NULL);
+       curl_easy_setopt(slot->curl, CURLOPT_POSTFIELDS, NULL);
        curl_easy_setopt(slot->curl, CURLOPT_UPLOAD, 0);
        curl_easy_setopt(slot->curl, CURLOPT_HTTPGET, 1);
  
@@@ -600,11 -559,9 +601,11 @@@ void fill_active_slots(void
        }
  
        while (slot != NULL) {
 -              if (!slot->in_use && slot->curl != NULL) {
 +              if (!slot->in_use && slot->curl != NULL
 +                      && curl_session_count > min_curl_sessions) {
                        curl_easy_cleanup(slot->curl);
                        slot->curl = NULL;
 +                      curl_session_count--;
                }
                slot = slot->next;
        }
@@@ -674,16 -631,15 +675,16 @@@ static void closedown_active_slot(struc
        slot->in_use = 0;
  }
  
 -void release_active_slot(struct active_request_slot *slot)
 +static void release_active_slot(struct active_request_slot *slot)
  {
        closedown_active_slot(slot);
 -      if (slot->curl) {
 +      if (slot->curl && curl_session_count > min_curl_sessions) {
  #ifdef USE_CURL_MULTI
                curl_multi_remove_handle(curlm, slot->curl);
  #endif
                curl_easy_cleanup(slot->curl);
                slot->curl = NULL;
 +              curl_session_count--;
        }
  #ifdef USE_CURL_MULTI
        fill_active_slots();
@@@ -743,6 -699,13 +744,6 @@@ static inline int hex(int v
                return 'A' + v - 10;
  }
  
 -static void end_url_with_slash(struct strbuf *buf, const char *url)
 -{
 -      strbuf_addstr(buf, url);
 -      if (buf->len && buf->buf[buf->len - 1] != '/')
 -              strbuf_addstr(buf, "/");
 -}
 -
  static char *quote_ref_url(const char *base, const char *ref)
  {
        struct strbuf buf = STRBUF_INIT;
@@@ -831,21 -794,7 +832,21 @@@ static int http_request(const char *url
                        ret = HTTP_OK;
                else if (missing_target(&results))
                        ret = HTTP_MISSING_TARGET;
 -              else
 +              else if (results.http_code == 401) {
 +                      if (user_name) {
 +                              ret = HTTP_NOAUTH;
 +                      } else {
 +                              /*
 +                               * git_getpass is needed here because its very likely stdin/stdout are
 +                               * pipes to our parent process.  So we instead need to use /dev/tty,
 +                               * but that is non-portable.  Using git_getpass() can at least be stubbed
 +                               * on other platforms with a different implementation if/when necessary.
 +                               */
 +                              user_name = xstrdup(git_getpass("Username: "));
 +                              init_curl_http_auth(slot->curl);
 +                              ret = HTTP_REAUTH;
 +                      }
 +              } else
                        ret = HTTP_ERROR;
        } else {
                error("Unable to start HTTP request for %s", url);
  
  int http_get_strbuf(const char *url, struct strbuf *result, int options)
  {
 -      return http_request(url, result, HTTP_REQUEST_STRBUF, options);
 +      int http_ret = http_request(url, result, HTTP_REQUEST_STRBUF, options);
 +      if (http_ret == HTTP_REAUTH) {
 +              http_ret = http_request(url, result, HTTP_REQUEST_STRBUF, options);
 +      }
 +      return http_ret;
  }
  
 -int http_get_file(const char *url, const char *filename, int options)
 +/*
 + * Downloads an url and stores the result in the given file.
 + *
 + * If a previous interrupted download is detected (i.e. a previous temporary
 + * file is still around) the download is resumed.
 + */
 +static int http_get_file(const char *url, const char *filename, int options)
  {
        int ret;
        struct strbuf tmpfile = STRBUF_INIT;
@@@ -930,67 -869,47 +931,67 @@@ int http_fetch_ref(const char *base, st
  }
  
  /* Helpers for fetching packs */
 -static int fetch_pack_index(unsigned char *sha1, const char *base_url)
 +static char *fetch_pack_index(unsigned char *sha1, const char *base_url)
  {
 -      int ret = 0;
 -      char *hex = xstrdup(sha1_to_hex(sha1));
 -      char *filename;
 -      char *url = NULL;
 +      char *url, *tmp;
        struct strbuf buf = STRBUF_INIT;
  
 -      if (has_pack_index(sha1)) {
 -              ret = 0;
 -              goto cleanup;
 -      }
 -
        if (http_is_verbose)
 -              fprintf(stderr, "Getting index for pack %s\n", hex);
 +              fprintf(stderr, "Getting index for pack %s\n", sha1_to_hex(sha1));
  
        end_url_with_slash(&buf, base_url);
 -      strbuf_addf(&buf, "objects/pack/pack-%s.idx", hex);
 +      strbuf_addf(&buf, "objects/pack/pack-%s.idx", sha1_to_hex(sha1));
        url = strbuf_detach(&buf, NULL);
  
 -      filename = sha1_pack_index_name(sha1);
 -      if (http_get_file(url, filename, 0) != HTTP_OK)
 -              ret = error("Unable to get pack index %s\n", url);
 +      strbuf_addf(&buf, "%s.temp", sha1_pack_index_name(sha1));
 +      tmp = strbuf_detach(&buf, NULL);
 +
 +      if (http_get_file(url, tmp, 0) != HTTP_OK) {
 +              error("Unable to get pack index %s\n", url);
 +              free(tmp);
 +              tmp = NULL;
 +      }
  
 -cleanup:
 -      free(hex);
        free(url);
 -      return ret;
 +      return tmp;
  }
  
  static int fetch_and_setup_pack_index(struct packed_git **packs_head,
        unsigned char *sha1, const char *base_url)
  {
        struct packed_git *new_pack;
 +      char *tmp_idx = NULL;
 +      int ret;
 +
 +      if (has_pack_index(sha1)) {
 +              new_pack = parse_pack_index(sha1, NULL);
 +              if (!new_pack)
 +                      return -1; /* parse_pack_index() already issued error message */
 +              goto add_pack;
 +      }
  
 -      if (fetch_pack_index(sha1, base_url))
 +      tmp_idx = fetch_pack_index(sha1, base_url);
 +      if (!tmp_idx)
                return -1;
  
 -      new_pack = parse_pack_index(sha1);
 -      if (!new_pack)
 +      new_pack = parse_pack_index(sha1, tmp_idx);
 +      if (!new_pack) {
 +              unlink(tmp_idx);
 +              free(tmp_idx);
 +
                return -1; /* parse_pack_index() already issued error message */
 +      }
 +
 +      ret = verify_pack_index(new_pack);
 +      if (!ret) {
 +              close_pack_index(new_pack);
 +              ret = move_temp_to_file(tmp_idx, sha1_pack_index_name(sha1));
 +      }
 +      free(tmp_idx);
 +      if (ret)
 +              return -1;
 +
 +add_pack:
        new_pack->next = *packs_head;
        *packs_head = new_pack;
        return 0;
@@@ -1054,62 -973,37 +1055,62 @@@ void release_http_pack_request(struct h
  
  int finish_http_pack_request(struct http_pack_request *preq)
  {
 -      int ret;
        struct packed_git **lst;
 +      struct packed_git *p = preq->target;
 +      char *tmp_idx;
 +      struct child_process ip;
 +      const char *ip_argv[8];
  
 -      preq->target->pack_size = ftell(preq->packfile);
 -
 -      if (preq->packfile != NULL) {
 -              fclose(preq->packfile);
 -              preq->packfile = NULL;
 -              preq->slot->local = NULL;
 -      }
 +      close_pack_index(p);
  
 -      ret = move_temp_to_file(preq->tmpfile, preq->filename);
 -      if (ret)
 -              return ret;
 +      fclose(preq->packfile);
 +      preq->packfile = NULL;
 +      preq->slot->local = NULL;
  
        lst = preq->lst;
 -      while (*lst != preq->target)
 +      while (*lst != p)
                lst = &((*lst)->next);
        *lst = (*lst)->next;
  
 -      if (verify_pack(preq->target))
 +      tmp_idx = xstrdup(preq->tmpfile);
 +      strcpy(tmp_idx + strlen(tmp_idx) - strlen(".pack.temp"),
 +             ".idx.temp");
 +
 +      ip_argv[0] = "index-pack";
 +      ip_argv[1] = "-o";
 +      ip_argv[2] = tmp_idx;
 +      ip_argv[3] = preq->tmpfile;
 +      ip_argv[4] = NULL;
 +
 +      memset(&ip, 0, sizeof(ip));
 +      ip.argv = ip_argv;
 +      ip.git_cmd = 1;
 +      ip.no_stdin = 1;
 +      ip.no_stdout = 1;
 +
 +      if (run_command(&ip)) {
 +              unlink(preq->tmpfile);
 +              unlink(tmp_idx);
 +              free(tmp_idx);
                return -1;
 -      install_packed_git(preq->target);
 +      }
  
 +      unlink(sha1_pack_index_name(p->sha1));
 +
 +      if (move_temp_to_file(preq->tmpfile, sha1_pack_name(p->sha1))
 +       || move_temp_to_file(tmp_idx, sha1_pack_index_name(p->sha1))) {
 +              free(tmp_idx);
 +              return -1;
 +      }
 +
 +      install_packed_git(p);
 +      free(tmp_idx);
        return 0;
  }
  
  struct http_pack_request *new_http_pack_request(
        struct packed_git *target, const char *base_url)
  {
 -      char *filename;
        long prev_posn = 0;
        char range[RANGE_HEADER_SIZE];
        struct strbuf buf = STRBUF_INIT;
                sha1_to_hex(target->sha1));
        preq->url = strbuf_detach(&buf, NULL);
  
 -      filename = sha1_pack_name(target->sha1);
 -      snprintf(preq->filename, sizeof(preq->filename), "%s", filename);
 -      snprintf(preq->tmpfile, sizeof(preq->tmpfile), "%s.temp", filename);
 +      snprintf(preq->tmpfile, sizeof(preq->tmpfile), "%s.temp",
 +              sha1_pack_name(target->sha1));
        preq->packfile = fopen(preq->tmpfile, "a");
        if (!preq->packfile) {
                error("Unable to open local file %s for pack",
        return preq;
  
  abort:
 -      free(filename);
        free(preq->url);
        free(preq);
        return NULL;
@@@ -1214,6 -1110,7 +1215,6 @@@ struct http_object_request *new_http_ob
        freq->localfile = -1;
  
        filename = sha1_file_name(sha1);
 -      snprintf(freq->filename, sizeof(freq->filename), "%s", filename);
        snprintf(freq->tmpfile, sizeof(freq->tmpfile),
                 "%s.temp", filename);
  
        }
  
        if (freq->localfile < 0) {
 -              error("Couldn't create temporary file %s for %s: %s",
 -                    freq->tmpfile, freq->filename, strerror(errno));
 +              error("Couldn't create temporary file %s: %s",
 +                    freq->tmpfile, strerror(errno));
                goto abort;
        }
  
                        prev_posn = 0;
                        lseek(freq->localfile, 0, SEEK_SET);
                        if (ftruncate(freq->localfile, 0) < 0) {
 -                              error("Couldn't truncate temporary file %s for %s: %s",
 -                                        freq->tmpfile, freq->filename, strerror(errno));
 +                              error("Couldn't truncate temporary file %s: %s",
 +                                        freq->tmpfile, strerror(errno));
                                goto abort;
                        }
                }
@@@ -1348,7 -1245,7 +1349,7 @@@ int finish_http_object_request(struct h
        process_http_object_request(freq);
  
        if (freq->http_code == 416) {
 -              fprintf(stderr, "Warning: requested range invalid; we may already have all the data.\n");
 +              warning("requested range invalid; we may already have all the data.");
        } else if (freq->curl_result != CURLE_OK) {
                if (stat(freq->tmpfile, &st) == 0)
                        if (st.st_size == 0)
                return -1;
        }
        freq->rename =
 -              move_temp_to_file(freq->tmpfile, freq->filename);
 +              move_temp_to_file(freq->tmpfile, sha1_file_name(freq->sha1));
  
        return freq->rename;
  }