From: Matthias Wachs <wachs@net.in.tum.de>
Date: Thu, 26 Aug 2010 14:33:03 +0000 (+0000)
Subject: FIX: bugs found by klocwork
X-Git-Tag: initial-import-from-subversion-38251~20530
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=a52fe978a15db0438ba5534e73724f6308d60330;p=oweals%2Fgnunet.git

FIX: bugs found by klocwork
---

diff --git a/src/transport/benchmark.sh b/src/transport/benchmark.sh
index 2ce07e655..6ae68af2a 100755
--- a/src/transport/benchmark.sh
+++ b/src/transport/benchmark.sh
@@ -1,12 +1,12 @@
 #/bin/sh
 
-for i in {1..10}
+for i in {1..0}
 do
  echo RUN $i
  ./test_transport_api_reliability_http
 done
 
-for i in {1..10}
+for i in {1..100}
 do
  echo RUN $i
  ./test_transport_api_reliability_https
diff --git a/src/transport/plugin_transport_http.c b/src/transport/plugin_transport_http.c
index 2293e4ba4..952078865 100644
--- a/src/transport/plugin_transport_http.c
+++ b/src/transport/plugin_transport_http.c
@@ -419,13 +419,13 @@ static char * create_url(void * cls, const void * addr, size_t addrlen, size_t i
 {
   struct Plugin *plugin = cls;
   char *url = NULL;
+  char *addr_str = (char *) http_plugin_address_to_string(NULL, addr, addrlen);
 
   GNUNET_assert ((addr!=NULL) && (addrlen != 0));
   GNUNET_asprintf(&url,
-                  "%s://%s/%s;%u", PROTOCOL_PREFIX,
-                  http_plugin_address_to_string(NULL, addr, addrlen),
+                  "%s://%s/%s;%u", PROTOCOL_PREFIX, addr_str,
                   (char *) (&plugin->my_ascii_hash_ident),id);
-
+  GNUNET_free_non_null(addr_str);
   return url;
 }
 
diff --git a/src/transport/plugin_transport_https.c b/src/transport/plugin_transport_https.c
index 088c8f3d8..ff87f3c94 100644
--- a/src/transport/plugin_transport_https.c
+++ b/src/transport/plugin_transport_https.c
@@ -429,13 +429,13 @@ static char * create_url(void * cls, const void * addr, size_t addrlen, size_t i
 {
   struct Plugin *plugin = cls;
   char *url = NULL;
+  char *addr_str =  (char *) http_plugin_address_to_string(NULL, addr, addrlen);
 
   GNUNET_assert ((addr!=NULL) && (addrlen != 0));
   GNUNET_asprintf(&url,
-                  "%s://%s/%s;%u", PROTOCOL_PREFIX,
-                  http_plugin_address_to_string(NULL, addr, addrlen),
+                  "%s://%s/%s;%u", PROTOCOL_PREFIX, addr_str,
                   (char *) (&plugin->my_ascii_hash_ident),id);
-
+  GNUNET_free_non_null(addr_str);
   return url;
 }
 
@@ -818,7 +818,7 @@ mdh_access_cb (void *cls,
 
   int res = GNUNET_NO;
   int send_error_to_client;
-  void * addr;
+  void * addr = NULL;
   size_t addr_len = 0;
 
   GNUNET_assert(cls !=NULL);
@@ -2294,6 +2294,7 @@ load_certificate( const char * file )
   if (GNUNET_SYSERR == GNUNET_DISK_file_read(gn_file, text, fstat.st_size))
   {
 	  GNUNET_free(text);
+	  GNUNET_DISK_file_close(gn_file);
 	  return NULL;
   }
   text[fstat.st_size] = '\0';
@@ -2367,6 +2368,8 @@ libgnunet_plugin_transport_https_done (void *cls)
   GNUNET_free_non_null (plugin->bind6_address);
   GNUNET_free_non_null (plugin->bind_hostname);
   GNUNET_free_non_null (plugin->crypto_init);
+  GNUNET_free_non_null (plugin->cert);
+  GNUNET_free_non_null (plugin->key);
   GNUNET_free (plugin);
   GNUNET_free (api);
 #if DEBUG_HTTPS
@@ -2388,8 +2391,8 @@ libgnunet_plugin_transport_https_init (void *cls)
   struct GNUNET_TIME_Relative gn_timeout;
   long long unsigned int port;
 
-  char * key_file;
-  char * cert_file;
+  char * key_file = NULL;
+  char * cert_file = NULL;
 
   GNUNET_assert(cls !=NULL);
 #if DEBUG_HTTPS
@@ -2487,20 +2490,6 @@ libgnunet_plugin_transport_https_init (void *cls)
 	  GNUNET_asprintf(&plugin->crypto_init,"NORMAL");
   }
 
-  /* Get private key file from config */
-  if (GNUNET_CONFIGURATION_have_value (env->cfg,
-		  	  	  	  	  	  	  	   "transport-https", "CERT_FILE"))
-  {
-	  GNUNET_CONFIGURATION_get_value_string (env->cfg,
-										   	 "transport-https",
-										     "CERT_FILE",
-										     &cert_file);
-  }
-  else
-  {
-	  GNUNET_asprintf(&cert_file,"https.cert");
-  }
-
   /* Get private key file from config */
   if (GNUNET_CONFIGURATION_have_value (env->cfg,
 		  	  	  	  	  	  	  	   "transport-https", "KEY_FILE"))
@@ -2510,24 +2499,19 @@ libgnunet_plugin_transport_https_init (void *cls)
 											   "KEY_FILE",
 											   &key_file);
   }
-  else
-  {
+  if (key_file==NULL)
 	  GNUNET_asprintf(&key_file,"https.key");
-  }
 
   /* Get private key file from config */
-  if (GNUNET_CONFIGURATION_have_value (env->cfg,
-		  	  	  	  	  	  	  	   "transport-https", "CERT_FILE"))
+  if (GNUNET_CONFIGURATION_have_value (env->cfg,"transport-https", "CERT_FILE"))
   {
 	  GNUNET_CONFIGURATION_get_value_string (env->cfg,
 										   	 "transport-https",
 										     "CERT_FILE",
 										     &cert_file);
   }
-  else
-  {
+  if (cert_file==NULL)
 	  GNUNET_asprintf(&cert_file,"https.cert");
-  }
 
   /* Should plugin use ipv6? */
   if ((plugin->use_ipv6==GNUNET_YES) && (GNUNET_CONFIGURATION_have_value (env->cfg,
@@ -2578,6 +2562,7 @@ libgnunet_plugin_transport_https_init (void *cls)
 						   "transport-https");
 		  GNUNET_free (key_file);
 		  GNUNET_free (cert_file);
+
 		  libgnunet_plugin_transport_https_done(api);
 		  GNUNET_free (cmd);
 		  return NULL;
@@ -2606,7 +2591,7 @@ libgnunet_plugin_transport_https_init (void *cls)
 
 
   GNUNET_assert((plugin->key!=NULL) && (plugin->cert!=NULL));
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "TLS certificate loaded\n", key_file, cert_file);
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "TLS certificate loaded\n");
 
   GNUNET_assert ((port > 0) && (port <= 65535));
   plugin->port_inbound = port;