summaryrefslogtreecommitdiff
path: root/src/bank-lib
diff options
context:
space:
mode:
authorChristian Grothoff <christian@grothoff.org>2020-03-03 16:35:20 +0100
committerChristian Grothoff <christian@grothoff.org>2020-03-03 16:35:20 +0100
commit97fb6b0091ca089b997febf691be2038223a8d21 (patch)
tree58b5a28660c007ea2716aed30de0fffaccfea928 /src/bank-lib
parent9abee82d3c9688549d50ec87153f6f3c1f973565 (diff)
downloadexchange-97fb6b0091ca089b997febf691be2038223a8d21.tar.gz
exchange-97fb6b0091ca089b997febf691be2038223a8d21.tar.bz2
exchange-97fb6b0091ca089b997febf691be2038223a8d21.zip
fixing memory leak, comments, renaming callback type to make it more concise
Diffstat (limited to 'src/bank-lib')
-rw-r--r--src/bank-lib/bank_api_admin.c26
-rw-r--r--src/bank-lib/bank_api_credit.c4
-rw-r--r--src/bank-lib/bank_api_debit.c4
-rw-r--r--src/bank-lib/bank_api_parse.c2
4 files changed, 18 insertions, 18 deletions
diff --git a/src/bank-lib/bank_api_admin.c b/src/bank-lib/bank_api_admin.c
index 979e0b06..0acc33ca 100644
--- a/src/bank-lib/bank_api_admin.c
+++ b/src/bank-lib/bank_api_admin.c
@@ -50,7 +50,7 @@ struct TALER_BANK_AdminAddIncomingHandle
/**
* Function to call with the result.
*/
- TALER_BANK_AdminAddIncomingResultCallback cb;
+ TALER_BANK_AdminAddIncomingCallback cb;
/**
* Closure for @a cb.
@@ -119,14 +119,13 @@ handle_admin_add_incoming_finished (void *cls,
ec = TALER_JSON_get_error_code (j);
break;
case MHD_HTTP_UNAUTHORIZED:
- /* Nothing really to verify, bank says one of the signatures is
- invalid; as we checked them, this should never happen, we
- should pass the JSON reply to the application */
+ /* Nothing really to verify, bank says the password is invalid; we should
+ pass the JSON reply to the application */
ec = TALER_JSON_get_error_code (j);
break;
case MHD_HTTP_NOT_FOUND:
- /* Nothing really to verify, this should never
- happen, we should pass the JSON reply to the application */
+ /* Nothing really to verify, maybe account really does not exist.
+ We should pass the JSON reply to the application */
ec = TALER_JSON_get_error_code (j);
break;
case MHD_HTTP_INTERNAL_SERVER_ERROR:
@@ -155,20 +154,20 @@ handle_admin_add_incoming_finished (void *cls,
/**
- * Notify the bank that we have received an incoming transaction
- * which fills a reserve. Note that this API is an administrative
- * API and thus not accessible to typical bank clients, but only
- * to the operators of the bank.
+ * Perform a wire transfer from some account to the exchange to fill a
+ * reserve. Note that this API is usually only used for testing (with
+ * fakebank and our Python bank) and thus may not be accessible in a
+ * production setting.
*
* @param ctx curl context for the event loop
* @param auth authentication data to send to the bank
* @param reserve_pub wire transfer subject for the transfer
- * @param amount amount that was deposited
+ * @param amount amount that was is to be deposited
* @param debit_account account to deposit from (payto URI, but used as 'payfrom')
* @param res_cb the callback to call when the final result for this request is available
* @param res_cb_cls closure for the above callback
* @return NULL
- * if the inputs are invalid (i.e. invalid amount).
+ * if the inputs are invalid (i.e. invalid amount) or internal errors.
* In this case, the callback is not called.
*/
struct TALER_BANK_AdminAddIncomingHandle *
@@ -178,7 +177,7 @@ TALER_BANK_admin_add_incoming (struct GNUNET_CURL_Context *ctx,
TALER_ReservePublicKeyP *reserve_pub,
const struct TALER_Amount *amount,
const char *debit_account,
- TALER_BANK_AdminAddIncomingResultCallback res_cb,
+ TALER_BANK_AdminAddIncomingCallback res_cb,
void *res_cb_cls)
{
struct TALER_BANK_AdminAddIncomingHandle *aai;
@@ -206,6 +205,7 @@ TALER_BANK_admin_add_incoming (struct GNUNET_CURL_Context *ctx,
if (NULL == aai->request_url)
{
GNUNET_free (aai);
+ json_decref (admin_obj);
GNUNET_break (0);
return NULL;
}
diff --git a/src/bank-lib/bank_api_credit.c b/src/bank-lib/bank_api_credit.c
index 66e128da..be5b1908 100644
--- a/src/bank-lib/bank_api_credit.c
+++ b/src/bank-lib/bank_api_credit.c
@@ -53,7 +53,7 @@ struct TALER_BANK_CreditHistoryHandle
/**
* Function to call with the result.
*/
- TALER_BANK_CreditResultCallback hcb;
+ TALER_BANK_CreditHistoryCallback hcb;
/**
* Closure for @a cb.
@@ -233,7 +233,7 @@ TALER_BANK_credit_history (struct GNUNET_CURL_Context *ctx,
const struct TALER_BANK_AuthenticationData *auth,
uint64_t start_row,
int64_t num_results,
- TALER_BANK_CreditResultCallback hres_cb,
+ TALER_BANK_CreditHistoryCallback hres_cb,
void *hres_cb_cls)
{
char *url;
diff --git a/src/bank-lib/bank_api_debit.c b/src/bank-lib/bank_api_debit.c
index 58f6ae6d..eec4a1df 100644
--- a/src/bank-lib/bank_api_debit.c
+++ b/src/bank-lib/bank_api_debit.c
@@ -48,7 +48,7 @@ struct TALER_BANK_DebitHistoryHandle
/**
* Function to call with the result.
*/
- TALER_BANK_DebitResultCallback hcb;
+ TALER_BANK_DebitHistoryCallback hcb;
/**
* Closure for @a cb.
@@ -232,7 +232,7 @@ TALER_BANK_debit_history (struct GNUNET_CURL_Context *ctx,
const struct TALER_BANK_AuthenticationData *auth,
uint64_t start_row,
int64_t num_results,
- TALER_BANK_DebitResultCallback hres_cb,
+ TALER_BANK_DebitHistoryCallback hres_cb,
void *hres_cb_cls)
{
char *url;
diff --git a/src/bank-lib/bank_api_parse.c b/src/bank-lib/bank_api_parse.c
index 8a50f5af..b09e5af3 100644
--- a/src/bank-lib/bank_api_parse.c
+++ b/src/bank-lib/bank_api_parse.c
@@ -123,7 +123,7 @@ TALER_BANK_auth_parse_cfg (const struct GNUNET_CONFIGURATION_Handle *cfg,
/**
- * Free memory inside of @a auth (but not auth itself).
+ * Free memory inside of @a auth (but not @a auth itself).
* Dual to #TALER_BANK_auth_parse_cfg().
*
* @param[in] auth authentication data to free