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 | |
| parent | 421f59749ec806a59bb2078c4f5c4dc0d52032dd (diff) | |
-improve error handling
Diffstat (limited to 'src')
| -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 f468c989..db30851d 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; | 
