[taler-exchange] branch master updated: -fix refund check regression

[gnunet]

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 f2e2c5c9 -fix refund check regression
f2e2c5c9 is described below

commit f2e2c5c9d81ede958bfdbddbab80d7a9a9ed4cb2
Author: Christian Grothoff <christian@grothoff.org>
AuthorDate: Tue Jun 7 22:24:49 2022 +0200

    -fix refund check regression
---
 src/lib/exchange_api_refund.c | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/src/lib/exchange_api_refund.c b/src/lib/exchange_api_refund.c
index b1cd7c84..2b0f5b03 100644
--- a/src/lib/exchange_api_refund.c
+++ b/src/lib/exchange_api_refund.c
@@ -444,20 +444,14 @@ verify_conflict_history_ok (struct 
TALER_EXCHANGE_RefundHandle *rh,
   else
   {
     rtotal = rh->refund_amount;
+    have_refund = true;
   }
-  if ( (have_refund) && (! have_deposit) )
+  if (! have_deposit)
   {
     GNUNET_break (0);
     GNUNET_JSON_parse_free (spec);
     return GNUNET_SYSERR;
   }
-  if (! (have_refund && have_deposit))
-  {
-    /* need both for a refund-deposit conflict proof */
-    GNUNET_break (0);
-    GNUNET_JSON_parse_free (spec);
-    return GNUNET_SYSERR;
-  }
   if (-1 != TALER_amount_cmp (&dtotal,
                               &rtotal))
   {
@@ -649,6 +643,9 @@ handle_refund_finished (void *cls,
                                     j))
     {
       GNUNET_break (0);
+      json_dumpf (j,
+                  stderr,
+                  JSON_INDENT (2));
       hr.http_status = 0;
       hr.ec = TALER_EC_EXCHANGE_REFUND_INVALID_FAILURE_PROOF_BY_EXCHANGE;
       hr.hint = "conflict information provided by exchange is invalid";

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