gnunet-svn
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[taler-exchange] branch master updated: -fix auditor test


From: gnunet
Subject: [taler-exchange] branch master updated: -fix auditor test
Date: Thu, 12 Dec 2024 09:17:59 +0100

This is an automated email from the git hooks/post-receive script.

grothoff pushed a commit to branch master
in repository exchange.

The following commit(s) were added to refs/heads/master by this push:
     new 43b75c8f1 -fix auditor test
43b75c8f1 is described below

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

    -fix auditor test
---
 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
index fd0d4835f..0b0e3c8d1 100644
--- 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 */

-- 
To stop receiving notification emails like this one, please contact
gnunet@gnunet.org.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]