exchange

commit 43b75c8f1ed12d07f2bebc1ee28810c7658dfb42
parent e2b004b38d75e69a6b12f842107e5778a51c4f42
Author: Christian Grothoff <christian@grothoff.org>
Date:   Thu, 12 Dec 2024 09:17:54 +0100

-fix auditor test

Diffstat:
M
src/auditor/taler-helper-auditor-aggregation.c
 | 
28
++++++++++++++--------------

1 file changed, 14 insertions(+), 14 deletions(-)
diff --git a/src/auditor/taler-helper-auditor-aggregation.c b/src/auditor/taler-helper-auditor-aggregation.c
@@ -1091,21 +1091,21 @@ get_wire_fee (struct AggregationContext *ac,
   /* Check signature. (This is not terribly meaningful as the exchange can
      easily make this one up, but it means that we have proof that the master
      key was used for inconsistent wire fees if a merchant complains.) */
+  if (GNUNET_OK !=
+      TALER_exchange_offline_wire_fee_verify (
+        method,
+        wfi->start_date,
+        wfi->end_date,
+        &wfi->fees,
+        &TALER_ARL_master_pub,
+        &master_sig))
   {
-    if (GNUNET_OK !=
-        TALER_exchange_offline_wire_fee_verify (
-          method,
-          wfi->start_date,
-          wfi->end_date,
-          &wfi->fees,
-          &TALER_ARL_master_pub,
-          &master_sig))
-    {
-      ac->qs = report_row_inconsistency ("wire-fee",
-                                         timestamp.abs_time.abs_value_us,
-                                         "wire fee signature invalid at given time");
-      return NULL;
-    }
+    ac->qs = report_row_inconsistency ("wire-fee",
+                                       timestamp.abs_time.abs_value_us,
+                                       "wire fee signature invalid at given time");
+    /* Note: continue with the fee despite the signature
+       being invalid here; hopefully it is really only the
+       signature that is bad ... */
   }
 
   /* Established fee, keep in sorted list */