From 8ca555500f01ebb2f7286961588b81b4ee5ac4fe Mon Sep 17 00:00:00 2001 From: Christian Grothoff Date: Thu, 29 Jan 2015 19:01:29 +0100 Subject: [PATCH] more notes about issues in bug tracker --- src/mint/taler-mint-httpd_db.c | 35 +++------------------- src/mint/taler-mint-httpd_refresh.c | 45 +++++++++++++++++++++++++++++ 2 files changed, 49 insertions(+), 31 deletions(-) diff --git a/src/mint/taler-mint-httpd_db.c b/src/mint/taler-mint-httpd_db.c index ac91cac5e..eae37199b 100644 --- a/src/mint/taler-mint-httpd_db.c +++ b/src/mint/taler-mint-httpd_db.c @@ -137,7 +137,7 @@ TALER_MINT_db_execute_deposit (struct MHD_Connection *connection, &deposit->coin.coin_pub); spent = fee_withdraw; /* fee for THIS transaction */ /* FIXME: need to deal better with integer overflows - in the logic that follows! (change amount.c API!) */ + in the logic that follows! (change amount.c API! -- #3637) */ spent = TALER_amount_add (spent, deposit->amount); @@ -164,7 +164,7 @@ TALER_MINT_db_execute_deposit (struct MHD_Connection *connection, is for THIS operation, ignore it; if lock is for another operation, count it! */ - GNUNET_assert (0); // FIXME: not implemented! + GNUNET_assert (0); // FIXME: not implemented! (#3625) break; } } @@ -440,8 +440,6 @@ TALER_MINT_db_execute_withdraw_sign (struct MHD_Connection *connection, * @param session_pub the refresh session public key * @param denom_pubs_count number of entries in @a denom_pubs * @param denom_pubs array of public keys for the refresh - * @param r_amount the sum of the cost (value+fee) for - * all requested coins * @return FIXME! */ static int @@ -450,24 +448,13 @@ refresh_accept_denoms (struct MHD_Connection *connection, const struct MintKeyState *key_state, const struct GNUNET_CRYPTO_EddsaPublicKey *session_pub, unsigned int denom_pubs_count, - struct GNUNET_CRYPTO_rsa_PublicKey *const*denom_pubs, - struct TALER_Amount *r_amount) + struct GNUNET_CRYPTO_rsa_PublicKey *const*denom_pubs) { unsigned int i; int res; - struct TALER_MINT_DenomKeyIssue *dki; - struct TALER_Amount cost; - memset (r_amount, 0, sizeof (struct TALER_Amount)); for (i = 0; i < denom_pubs_count; i++) { - dki = &(TALER_MINT_get_denom_key (key_state, - denom_pubs[i])->issue); - cost = TALER_amount_add (TALER_amount_ntoh (dki->value), - TALER_amount_ntoh (dki->fee_withdraw)); - *r_amount = TALER_amount_add (cost, *r_amount); - - /* Insert the requested coin into the DB, so we'll know later * what denomination the request had */ @@ -629,7 +616,6 @@ TALER_MINT_db_execute_refresh_melt (struct MHD_Connection *connection, unsigned int coin_count, const struct TALER_CoinPublicInfo *coin_public_infos) { - struct TALER_Amount requested_cost; struct TALER_Amount melt_balance; struct MintKeyState *key_state; struct RefreshSession session; @@ -697,8 +683,7 @@ TALER_MINT_db_execute_refresh_melt (struct MHD_Connection *connection, (res = refresh_accept_denoms (connection, db_conn, key_state, refresh_session_pub, num_new_denoms, - denom_pubs, - &requested_cost))) + denom_pubs))) { TALER_MINT_key_state_release (key_state); TALER_MINT_DB_rollback (db_conn); @@ -724,18 +709,6 @@ TALER_MINT_db_execute_refresh_melt (struct MHD_Connection *connection, /* Request is only ok if cost of requested coins * does not exceed value of melted coins. */ - // FIXME: also, consider fees? - if (TALER_amount_cmp (melt_balance, requested_cost) < 0) - { - TALER_MINT_DB_rollback (db_conn); - - return TALER_MINT_reply_json_pack (connection, - MHD_HTTP_FORBIDDEN, - "{s:s}", - "error", - "not enough coins melted"); - } - if (GNUNET_OK != TALER_MINT_DB_commit (db_conn)) { diff --git a/src/mint/taler-mint-httpd_refresh.c b/src/mint/taler-mint-httpd_refresh.c index 8bc8e7783..e7497327b 100644 --- a/src/mint/taler-mint-httpd_refresh.c +++ b/src/mint/taler-mint-httpd_refresh.c @@ -167,6 +167,12 @@ handle_refresh_melt_binary (struct MHD_Connection *connection, struct RefreshMeltSignatureBody body; char *buf; size_t buf_size; + struct MintKeyState *key_state; + struct TALER_MINT_DenomKeyIssue *dki; + struct TALER_Amount cost; + struct TALER_Amount total_cost; + struct TALER_Amount melt; + struct TALER_Amount total_melt; /* check that signature from the session public key is ok */ hash_context = GNUNET_CRYPTO_hash_context_start (); @@ -197,6 +203,45 @@ handle_refresh_melt_binary (struct MHD_Connection *connection, &body.purpose))) return (GNUNET_SYSERR == res) ? MHD_NO : MHD_YES; + // FIXME: badness, use proper way to set to zero... + key_state = TALER_MINT_key_state_acquire (); + memset (&total_cost, 0, sizeof (struct TALER_Amount)); + for (i=0;iissue; + cost = TALER_amount_add (TALER_amount_ntoh (dki->value), + TALER_amount_ntoh (dki->fee_withdraw)); + // FIXME: #3637 + total_cost = TALER_amount_add (cost, + total_cost); + } + + // FIXME: badness, use proper way to set to zero... + memset (&total_melt, 0, sizeof (struct TALER_Amount)); + for (i=0;i