exchange

commit 8e391e8bfb0cd37181de8d1a921855ec798eaaad
parent cce67665f702fe10df64c1bad6a1bf6cf19c06ab
Author: Florian Dold <florian.dold@gmail.com>
Date:   Wed,  1 May 2019 04:00:29 +0200

Don't free null strings.

When running in client-only mode, this string will never be allocated,
but the code tried to free it ...

Diffstat:
M
src/benchmark/taler-exchange-benchmark.c
 | 
8
++++----

1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/benchmark/taler-exchange-benchmark.c b/src/benchmark/taler-exchange-benchmark.c
@@ -816,8 +816,8 @@ int
 main (int argc,
       char *const *argv)
 {
-  char *exchange_url;
-  char *auditor_url;
+  char *exchange_url = NULL;
+  char *auditor_url = NULL;
   struct GNUNET_CONFIGURATION_Handle *cfg;
   struct GNUNET_GETOPT_CommandLineOption options[] = {
     GNUNET_GETOPT_option_mandatory
@@ -1031,8 +1031,8 @@ main (int argc,
                                NULL,
                                cfg_filename,
                                exchange_url);
-  GNUNET_free (exchange_url);
-  GNUNET_free (auditor_url);
+  GNUNET_free_non_null (exchange_url);
+  GNUNET_free_non_null (auditor_url);
   duration = GNUNET_TIME_absolute_get_duration (start_time);
   if (GNUNET_OK == result)
   {