From 5fc2e72a2a42434495237e7551af8ec054718171 Mon Sep 17 00:00:00 2001
From: Christian Grothoff <christian@grothoff.org>
Date: Tue, 3 Aug 2021 14:41:42 +0200
Subject: fix #6969

---
 src/bank-lib/taler-exchange-wire-gateway-client.c | 21 ++++++++++++++++-----
 src/util/url.c                                    |  3 ++-
 2 files changed, 18 insertions(+), 6 deletions(-)

(limited to 'src')

diff --git a/src/bank-lib/taler-exchange-wire-gateway-client.c b/src/bank-lib/taler-exchange-wire-gateway-client.c
index 14af7fb2c..842778088 100644
--- a/src/bank-lib/taler-exchange-wire-gateway-client.c
+++ b/src/bank-lib/taler-exchange-wire-gateway-client.c
@@ -606,7 +606,7 @@ run (void *cls,
          (NULL != auth.details.basic.password) )
     {
       fprintf (stderr,
-               "Conflicting authentication options provided. Please only use one method.\n");
+               "Error: Conflicting authentication options provided. Please only use one method.\n");
       GNUNET_SCHEDULER_shutdown ();
       return;
     }
@@ -616,7 +616,7 @@ run (void *cls,
                                    &auth))
     {
       fprintf (stderr,
-               "Authentication information not found in configuration section `%s'\n",
+               "Error: Authentication information not found in configuration section `%s'\n",
                account_section);
       GNUNET_SCHEDULER_shutdown ();
       return;
@@ -633,16 +633,27 @@ run (void *cls,
     else if (NULL == auth.wire_gateway_url)
     {
       fprintf (stderr,
-               "No account specified (use -b or -s options).\n");
+               "Error: No account specified (use -b or -s options).\n");
       GNUNET_SCHEDULER_shutdown ();
       return;
     }
   }
+  if ( (NULL == auth.wire_gateway_url) ||
+       (0 == strlen (auth.wire_gateway_url)) ||
+       (0 != strcasecmp ("http",
+                         auth.wire_gateway_url)) )
+  {
+    fprintf (stderr,
+             "Error: Invalid wire gateway URL `%s' configured.\n",
+             auth.wire_gateway_url);
+    GNUNET_SCHEDULER_shutdown ();
+    return;
+  }
   if ( (GNUNET_YES == incoming_history) &&
        (GNUNET_YES == outgoing_history) )
   {
     fprintf (stderr,
-             "Please specify only -i or -o, but not both.\n");
+             "Error: Please specify only -i or -o, but not both.\n");
     GNUNET_SCHEDULER_shutdown ();
     return;
   }
@@ -675,7 +686,7 @@ run (void *cls,
 
 
 /**
- * The main function of the taler-bank-transfer tool
+ * The main function of the taler-exchange-wire-gateway-client
  *
  * @param argc number of arguments from the command line
  * @param argv command line arguments
diff --git a/src/util/url.c b/src/util/url.c
index 3fbc4a048..cae36299f 100644
--- a/src/util/url.c
+++ b/src/util/url.c
@@ -240,7 +240,8 @@ TALER_url_join (const char *base_url,
   if (0 == strlen (base_url))
   {
     /* base URL can't be empty */
-    GNUNET_break (0);
+    GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
+                "Empty base URL specified\n");
     return NULL;
   }
   if ('/' != base_url[strlen (base_url) - 1])
-- 
cgit v1.2.3