curl tool: reorder free_config_fields() field handling

Reorder handling of fields to match same order as the one given by current
definition order of 'Configurable' struct fields. Fields currently not handled
marked for further inspection.

diff --git a/src/tool_cfgable.c b/src/tool_cfgable.c
index a666951ec05252f385c975973ebfc8c5d73dad1c..c2cfac67392f839de238dcd51bae010d000d6fb9 100644 (file)
--- a/src/tool_cfgable.c
+++ b/src/tool_cfgable.c
@@ -27,72 +27,99 @@
 
 #include "memdebug.h" /* keep this as LAST include */
 
-/* TODO: review that all dinamically allocated fields are free'd */
+/* TODO: review handling of unhandled fields marked below */
 
 void free_config_fields(struct Configurable *config)
 {
+  if(config->easy) {
+    curl_easy_cleanup(config->easy);
+    config->easy = NULL;
+  }
+
   Curl_safefree(config->random_file);
   Curl_safefree(config->egd_file);
-  Curl_safefree(config->trace_dump);
-  Curl_safefree(config->cipher_list);
-  Curl_safefree(config->userpwd);
-  Curl_safefree(config->postfields);
-  Curl_safefree(config->proxy);
-  Curl_safefree(config->proxyuserpwd);
-  Curl_safefree(config->noproxy);
+
+  /* config->useragent not handled */
+
   Curl_safefree(config->cookie);
+  Curl_safefree(config->cookiejar);
   Curl_safefree(config->cookiefile);
-  Curl_safefree(config->krblevel);
+
+  Curl_safefree(config->postfields);
+  Curl_safefree(config->referer);
+
   Curl_safefree(config->headerfile);
   Curl_safefree(config->ftpport);
+  Curl_safefree(config->iface);
+
   Curl_safefree(config->range);
-  Curl_safefree(config->customrequest);
-  Curl_safefree(config->writeout);
 
-  if(config->httppost) {
-    curl_formfree(config->httppost);
-    config->httppost = NULL;
-  }
+  Curl_safefree(config->userpwd);
+  Curl_safefree(config->tls_username);
+  Curl_safefree(config->tls_password);
+  Curl_safefree(config->tls_authtype);
+  Curl_safefree(config->proxyuserpwd);
+  Curl_safefree(config->proxy);
+
+  Curl_safefree(config->noproxy);
+  Curl_safefree(config->mail_from);
+  curl_slist_free_all(config->mail_rcpt);
 
   Curl_safefree(config->netrc_file);
+
+  /* config->url_list not handled */
+  /* config->url_last not handled */
+  /* config->url_get not handled */
+  /* config->url_out not handled */
+
+  Curl_safefree(config->cipher_list);
   Curl_safefree(config->cert);
-  Curl_safefree(config->cacert);
   Curl_safefree(config->cert_type);
+  Curl_safefree(config->cacert);
   Curl_safefree(config->capath);
   Curl_safefree(config->crlfile);
-  Curl_safefree(config->cookiejar);
-  Curl_safefree(config->ftp_account);
-  Curl_safefree(config->ftp_alternative_to_user);
-  Curl_safefree(config->iface);
-  Curl_safefree(config->socksproxy);
-  Curl_safefree(config->libcurl);
-  Curl_safefree(config->key_passwd);
   Curl_safefree(config->key);
   Curl_safefree(config->key_type);
+  Curl_safefree(config->key_passwd);
   Curl_safefree(config->pubkey);
-  Curl_safefree(config->referer);
   Curl_safefree(config->hostpubmd5);
-  Curl_safefree(config->mail_from);
 
-#ifdef USE_TLS_SRP
-  Curl_safefree(config->tls_authtype);
-  Curl_safefree(config->tls_username);
-  Curl_safefree(config->tls_password);
-#endif
-#if defined(HAVE_GSSAPI) || defined(USE_WINDOWS_SSPI)
-  Curl_safefree(config->socks5_gssapi_service);
-#endif
+  /* config->engine not handled */
 
-  curl_slist_free_all(config->quote); /* checks for config->quote == NULL */
-  curl_slist_free_all(config->prequote);
+  Curl_safefree(config->customrequest);
+  Curl_safefree(config->krblevel);
+  Curl_safefree(config->trace_dump);
+
+  /* config->trace_stream not handled */
+
+  Curl_safefree(config->writeout);
+
+  /* config->errors not handled */
+
+  curl_slist_free_all(config->quote);
   curl_slist_free_all(config->postquote);
+  curl_slist_free_all(config->prequote);
+
   curl_slist_free_all(config->headers);
+
+  if(config->httppost) {
+    curl_formfree(config->httppost);
+    config->httppost = NULL;
+  }
+
+  /* config->last_post not handled */
+
   curl_slist_free_all(config->telnet_options);
-  curl_slist_free_all(config->mail_rcpt);
   curl_slist_free_all(config->resolve);
 
-  if(config->easy) {
-    curl_easy_cleanup(config->easy);
-    config->easy = NULL;
-  }
+  Curl_safefree(config->socksproxy);
+  Curl_safefree(config->socks5_gssapi_service);
+
+  Curl_safefree(config->ftp_account);
+  Curl_safefree(config->ftp_alternative_to_user);
+
+  Curl_safefree(config->libcurl);
+
+  /* config->outs not handled */
+
 }