diff options
author | Christian Grothoff <christian@grothoff.org> | 2022-05-21 01:39:34 +0200 |
---|---|---|
committer | Christian Grothoff <christian@grothoff.org> | 2022-05-21 01:39:34 +0200 |
commit | 81ad77b8dc6a90129ece7ef2c391aca83632d53a (patch) | |
tree | eee64ee372d5ffccbcd7e3791b3998703eb7b8ef /src/bank-lib/fakebank.c | |
parent | 421f59749ec806a59bb2078c4f5c4dc0d52032dd (diff) | |
download | exchange-81ad77b8dc6a90129ece7ef2c391aca83632d53a.tar.xz |
-improve error handling
Diffstat (limited to 'src/bank-lib/fakebank.c')
-rw-r--r-- | src/bank-lib/fakebank.c | 20 |
1 files changed, 19 insertions, 1 deletions
diff --git a/src/bank-lib/fakebank.c b/src/bank-lib/fakebank.c index f468c989c..db30851d8 100644 --- a/src/bank-lib/fakebank.c +++ b/src/bank-lib/fakebank.c @@ -1440,6 +1440,15 @@ handle_admin_add_incoming (struct TALER_FAKEBANK_Handle *h, NULL); } debit = TALER_xtalerbank_account_from_payto (debit_account); + if (NULL == debit) + { + GNUNET_break_op (0); + return TALER_MHD_reply_with_error ( + connection, + MHD_HTTP_BAD_REQUEST, + TALER_EC_GENERIC_PAYTO_URI_MALFORMED, + debit_account); + } GNUNET_log (GNUNET_ERROR_TYPE_INFO, "Receiving incoming wire transfer: %s->%s, subject: %s, amount: %s\n", debit, @@ -1560,6 +1569,15 @@ handle_transfer (struct TALER_FAKEBANK_Handle *h, int ret; credit = TALER_xtalerbank_account_from_payto (credit_account); + if (NULL == credit) + { + GNUNET_break_op (0); + return TALER_MHD_reply_with_error ( + connection, + MHD_HTTP_BAD_REQUEST, + TALER_EC_GENERIC_PAYTO_URI_MALFORMED, + credit_account); + } ret = make_transfer (h, account, credit, @@ -1649,7 +1667,7 @@ struct HistoryArgs uint64_t account_number; /** - * Index of the starting transaction. + * Index of the starting transaction, exclusive (!). */ uint64_t start_idx; |