[taler-exchange] branch master updated: use global ENABLE_SANITY_CHECKS constant also in response generation logic

[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 13eed90b use global ENABLE_SANITY_CHECKS constant also in response 
generation logic
13eed90b is described below

commit 13eed90b733553ae6df8e1af51ef410d8f92b364
Author: Christian Grothoff <address@hidden>
AuthorDate: Sun Mar 1 13:53:56 2020 +0100

    use global ENABLE_SANITY_CHECKS constant also in response generation logic
---
 src/exchange/taler-exchange-httpd_responses.c | 14 +++-----------
 src/mhd/mhd_responses.c                       |  9 ---------
 2 files changed, 3 insertions(+), 20 deletions(-)

diff --git a/src/exchange/taler-exchange-httpd_responses.c 
b/src/exchange/taler-exchange-httpd_responses.c
index 90ca14c8..9795d61a 100644
--- a/src/exchange/taler-exchange-httpd_responses.c
+++ b/src/exchange/taler-exchange-httpd_responses.c
@@ -30,14 +30,6 @@
 #include "taler_mhd_lib.h"
 #include "taler-exchange-httpd_keystate.h"
 
-/**
- * Enable checking signatures before we hand them out
- * (even though we should have checked them before).
- * So technically these checks are redundant, but good
- * during testing.
- */
-#define SANITY_CHECKS_ON 1
-
 
 /**
  * Compile the transaction history of a coin into a JSON object.
@@ -85,7 +77,7 @@ TEH_RESPONSE_compile_transaction_history (const struct
                            &deposit->deposit_fee);
         dr.merchant = deposit->merchant_pub;
         dr.coin_pub = *coin_pub;
-#if SANITY_CHECKS_ON
+#if ENABLE_SANITY_CHECKS
         /* internal sanity check before we hand out a bogus sig... */
         if (GNUNET_OK !=
             GNUNET_CRYPTO_eddsa_verify (TALER_SIGNATURE_WALLET_COIN_DEPOSIT,
@@ -148,7 +140,7 @@ TEH_RESPONSE_compile_transaction_history (const struct
         TALER_amount_hton (&ms.melt_fee,
                            &melt->melt_fee);
         ms.coin_pub = *coin_pub;
-#if SANITY_CHECKS_ON
+#if ENABLE_SANITY_CHECKS
         /* internal sanity check before we hand out a bogus sig... */
         if (GNUNET_OK !=
             GNUNET_CRYPTO_eddsa_verify (TALER_SIGNATURE_WALLET_COIN_MELT,
@@ -208,7 +200,7 @@ TEH_RESPONSE_compile_transaction_history (const struct
                            &refund->refund_amount);
         TALER_amount_hton (&rr.refund_fee,
                            &refund->refund_fee);
-#if SANITY_CHECKS_ON
+#if ENABLE_SANITY_CHECKS
         /* internal sanity check before we hand out a bogus sig... */
         if (GNUNET_OK !=
             GNUNET_CRYPTO_eddsa_verify (TALER_SIGNATURE_MERCHANT_REFUND,
diff --git a/src/mhd/mhd_responses.c b/src/mhd/mhd_responses.c
index 6e518148..05f95b7f 100644
--- a/src/mhd/mhd_responses.c
+++ b/src/mhd/mhd_responses.c
@@ -26,15 +26,6 @@
 #include "taler_mhd_lib.h"
 
 
-/**
- * Enable checking signatures before we hand them out
- * (even though we should have checked them before).
- * So technically these checks are redundant, but good
- * during testing.
- */
-#define SANITY_CHECKS_ON 1
-
-
 /**
  * Global options for response generation.
  */

-- 
To stop receiving notification emails like this one, please contact
address@hidden.