diff options
author | Christian Grothoff <christian@grothoff.org> | 2020-03-01 17:51:46 +0100 |
---|---|---|
committer | Christian Grothoff <christian@grothoff.org> | 2020-03-01 17:51:46 +0100 |
commit | 113f3891d13a15b87827a8b1b33508231a1409b7 (patch) | |
tree | 8b665c481282b0d3778deee534fd2b36b992608c /src/exchange/taler-exchange-aggregator.c | |
parent | c11475c081afcd5ccc667451b9383510fd42cf1b (diff) | |
download | exchange-113f3891d13a15b87827a8b1b33508231a1409b7.tar.gz exchange-113f3891d13a15b87827a8b1b33508231a1409b7.tar.bz2 exchange-113f3891d13a15b87827a8b1b33508231a1409b7.zip |
removing dead code, using more named constants, checking return values better
Diffstat (limited to 'src/exchange/taler-exchange-aggregator.c')
-rw-r--r-- | src/exchange/taler-exchange-aggregator.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/exchange/taler-exchange-aggregator.c b/src/exchange/taler-exchange-aggregator.c index 1a8889fda..bf066d846 100644 --- a/src/exchange/taler-exchange-aggregator.c +++ b/src/exchange/taler-exchange-aggregator.c @@ -405,6 +405,7 @@ update_fees (struct WireAccount *wa, &p->master_sig); if (qs < 0) { + GNUNET_break (GNUNET_DB_STATUS_SOFT_ERROR == qs); TALER_EXCHANGEDB_fees_free (wa->af); wa->af = NULL; return NULL; @@ -854,10 +855,9 @@ deposit_cb (void *cls, au->session); if (NULL == af) { - if (GNUNET_DB_STATUS_SUCCESS_NO_RESULTS == qs) - qs = GNUNET_DB_STATUS_HARD_ERROR; - GNUNET_break (GNUNET_DB_STATUS_SOFT_ERROR == qs); - return qs; + GNUNET_log (GNUNET_ERROR_TYPE_ERROR, + "Could not get or persist wire fees. Aborting run.\n"); + return GNUNET_DB_STATUS_HARD_ERROR; } au->wire_fee = af->wire_fee; } |