summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChristian Grothoff <christian@grothoff.org>2020-03-01 13:53:56 +0100
committerChristian Grothoff <christian@grothoff.org>2020-03-01 13:53:56 +0100
commit13eed90b733553ae6df8e1af51ef410d8f92b364 (patch)
tree7fe4f320e185cace2033813265910adc9a4ecd7a
parent9b688d7cb096a3a8cd30d57384eb00eb5c42b4c5 (diff)
downloadexchange-13eed90b733553ae6df8e1af51ef410d8f92b364.tar.gz
exchange-13eed90b733553ae6df8e1af51ef410d8f92b364.tar.bz2
exchange-13eed90b733553ae6df8e1af51ef410d8f92b364.zip
use global ENABLE_SANITY_CHECKS constant also in response generation logic
-rw-r--r--src/exchange/taler-exchange-httpd_responses.c14
-rw-r--r--src/mhd/mhd_responses.c9
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 90ca14c88..9795d61af 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 6e5181482..05f95b7f1 100644
--- a/src/mhd/mhd_responses.c
+++ b/src/mhd/mhd_responses.c
@@ -27,15 +27,6 @@
/**
- * 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.
*/
static enum TALER_MHD_GlobalOptions TM_go;