Merge branch 'master' of taler.net:/var/git/mint
This commit is contained in:
commit
b77bb5caa0
@ -476,9 +476,9 @@ struct TALER_MINT_ReserveHistory
|
||||
/**
|
||||
* Signature authorizing the withdrawal for outgoing transaction.
|
||||
*/
|
||||
struct TALER_ReserveSignatureP out_authorization_sig;
|
||||
json_t *out_authorization_sig;
|
||||
|
||||
};
|
||||
} details;
|
||||
|
||||
};
|
||||
|
||||
@ -490,13 +490,15 @@ struct TALER_MINT_ReserveHistory
|
||||
* @param cls closure
|
||||
* @param http_status HTTP response code, #MHD_HTTP_OK (200) for successful status request
|
||||
* 0 if the mint's reply is bogus (fails to follow the protocol)
|
||||
* @param balance current balance in the reserve
|
||||
* @param history_length number of entries in the transaction history
|
||||
* @param history detailed transaction history
|
||||
* @param[in] json original response in JSON format (useful only for diagnostics)
|
||||
* @param balance current balance in the reserve, NULL on error
|
||||
* @param history_length number of entries in the transaction history, 0 on error
|
||||
* @param history detailed transaction history, NULL on error
|
||||
*/
|
||||
typedef void
|
||||
(*TALER_MINT_WithdrawStatusResultCallback) (void *cls,
|
||||
unsigned int http_status,
|
||||
json_t *json,
|
||||
const struct TALER_Amount *balance,
|
||||
unsigned int history_length,
|
||||
const struct TALER_MINT_ReserveHistory *history);
|
||||
|
@ -68,6 +68,11 @@ struct TALER_MINT_WithdrawStatusHandle
|
||||
*/
|
||||
TALER_MINT_WithdrawStatusResultCallback cb;
|
||||
|
||||
/**
|
||||
* Public key of the reserve we are querying.
|
||||
*/
|
||||
struct TALER_ReservePublicKeyP reserve_pub;
|
||||
|
||||
/**
|
||||
* Closure for @a cb.
|
||||
*/
|
||||
@ -92,6 +97,191 @@ struct TALER_MINT_WithdrawStatusHandle
|
||||
};
|
||||
|
||||
|
||||
/**
|
||||
* Parse history given in JSON format and return it in binary
|
||||
* format.
|
||||
*
|
||||
* @param[in] history JSON array with the history
|
||||
* @param reserve_pub public key of the reserve to inspect
|
||||
* @param currency currency we expect the balance to be in
|
||||
* @param[out] balance final balance
|
||||
* @param history_length number of entries in @a history
|
||||
* @param[out] rhistory array of length @a history_length, set to the
|
||||
* parsed history entries
|
||||
* @return #GNUNET_OK if history was valid and @a rhistory and @a balance
|
||||
* were set,
|
||||
* #GNUNET_SYSERR if there was a protocol violation in @a history
|
||||
*/
|
||||
static int
|
||||
parse_reserve_history (json_t *history,
|
||||
const struct TALER_ReservePublicKeyP *reserve_pub,
|
||||
const char *currency,
|
||||
struct TALER_Amount *balance,
|
||||
unsigned int history_length,
|
||||
struct TALER_MINT_ReserveHistory *rhistory)
|
||||
{
|
||||
struct GNUNET_HashCode uuid[history_length];
|
||||
unsigned int uuid_off;
|
||||
struct TALER_Amount total_in;
|
||||
struct TALER_Amount total_out;
|
||||
size_t off;
|
||||
|
||||
TALER_amount_get_zero (currency,
|
||||
&total_in);
|
||||
TALER_amount_get_zero (currency,
|
||||
&total_out);
|
||||
uuid_off = 0;
|
||||
for (off=0;off<history_length;off++)
|
||||
{
|
||||
json_t *transaction;
|
||||
struct TALER_Amount amount;
|
||||
const char *type;
|
||||
struct MAJ_Specification hist_spec[] = {
|
||||
MAJ_spec_string ("type", &type),
|
||||
MAJ_spec_amount ("amount",
|
||||
&amount),
|
||||
/* 'wire' and 'signature' are optional depending on 'type'! */
|
||||
MAJ_spec_end
|
||||
};
|
||||
|
||||
transaction = json_array_get (history,
|
||||
off);
|
||||
if (GNUNET_OK !=
|
||||
MAJ_parse_json (transaction,
|
||||
hist_spec))
|
||||
{
|
||||
GNUNET_break_op (0);
|
||||
return GNUNET_SYSERR;
|
||||
}
|
||||
rhistory[off].amount = amount;
|
||||
|
||||
if (0 == strcasecmp (type,
|
||||
"DEPOSIT"))
|
||||
{
|
||||
json_t *wire;
|
||||
|
||||
rhistory[off].type = TALER_MINT_RTT_DEPOSIT;
|
||||
if (GNUNET_OK !=
|
||||
TALER_amount_add (&total_in,
|
||||
&total_in,
|
||||
&amount))
|
||||
{
|
||||
/* overflow in history already!? inconceivable! Bad mint! */
|
||||
GNUNET_break_op (0);
|
||||
return GNUNET_SYSERR;
|
||||
}
|
||||
wire = json_object_get (transaction,
|
||||
"wire");
|
||||
/* check 'wire' is a JSON object (no need to check wireformat,
|
||||
but we do at least expect "some" JSON object here) */
|
||||
if ( (NULL == wire) ||
|
||||
(! json_is_object (wire)) )
|
||||
{
|
||||
/* not even a JSON 'wire' specification, not acceptable */
|
||||
GNUNET_break_op (0);
|
||||
return GNUNET_SYSERR;
|
||||
}
|
||||
rhistory[off].details.wire_in_details = wire;
|
||||
/* end type==DEPOSIT */
|
||||
}
|
||||
else if (0 == strcasecmp (type,
|
||||
"WITHDRAW"))
|
||||
{
|
||||
struct GNUNET_CRYPTO_EccSignaturePurpose *purpose;
|
||||
const struct TALER_WithdrawRequestPS *withdraw_purpose;
|
||||
struct TALER_Amount amount_from_purpose;
|
||||
struct MAJ_Specification withdraw_spec[] = {
|
||||
MAJ_spec_eddsa_signed_purpose ("signature",
|
||||
&purpose,
|
||||
&reserve_pub->eddsa_pub),
|
||||
MAJ_spec_end
|
||||
};
|
||||
unsigned int i;
|
||||
|
||||
rhistory[off].type = TALER_MINT_RTT_WITHDRAWAL;
|
||||
if (GNUNET_OK !=
|
||||
MAJ_parse_json (transaction,
|
||||
withdraw_spec))
|
||||
{
|
||||
GNUNET_break_op (0);
|
||||
return GNUNET_SYSERR;
|
||||
}
|
||||
/* Check that the signature actually signed a withdraw request */
|
||||
if ( (ntohl (purpose->purpose) != TALER_SIGNATURE_WALLET_RESERVE_WITHDRAW) ||
|
||||
(ntohl (purpose->size) != sizeof (struct TALER_WithdrawRequestPS)) )
|
||||
{
|
||||
GNUNET_break_op (0);
|
||||
MAJ_parse_free (withdraw_spec);
|
||||
return GNUNET_SYSERR;
|
||||
}
|
||||
withdraw_purpose = (const struct TALER_WithdrawRequestPS *) purpose;
|
||||
TALER_amount_ntoh (&amount_from_purpose,
|
||||
&withdraw_purpose->amount_with_fee);
|
||||
if (0 != TALER_amount_cmp (&amount,
|
||||
&amount_from_purpose))
|
||||
{
|
||||
GNUNET_break_op (0);
|
||||
MAJ_parse_free (withdraw_spec);
|
||||
return GNUNET_SYSERR;
|
||||
}
|
||||
rhistory[off].details.out_authorization_sig = json_object_get (transaction,
|
||||
"signature");
|
||||
/* Check check that the same withdraw transaction
|
||||
isn't listed twice by the mint. We use the
|
||||
"uuid" array to remember the hashes of all
|
||||
purposes, and compare the hashes to find
|
||||
duplicates. */
|
||||
GNUNET_CRYPTO_hash (withdraw_purpose,
|
||||
ntohl (withdraw_purpose->purpose.size),
|
||||
&uuid[uuid_off]);
|
||||
for (i=0;i<uuid_off;i++)
|
||||
{
|
||||
if (0 == memcmp (&uuid[uuid_off],
|
||||
&uuid[i],
|
||||
sizeof (struct GNUNET_HashCode)))
|
||||
{
|
||||
GNUNET_break_op (0);
|
||||
MAJ_parse_free (withdraw_spec);
|
||||
return GNUNET_SYSERR;
|
||||
}
|
||||
}
|
||||
uuid_off++;
|
||||
|
||||
if (GNUNET_OK !=
|
||||
TALER_amount_add (&total_out,
|
||||
&total_out,
|
||||
&amount))
|
||||
{
|
||||
/* overflow in history already!? inconceivable! Bad mint! */
|
||||
GNUNET_break_op (0);
|
||||
MAJ_parse_free (withdraw_spec);
|
||||
return GNUNET_SYSERR;
|
||||
}
|
||||
/* end type==WITHDRAW */
|
||||
}
|
||||
else
|
||||
{
|
||||
/* unexpected 'type', protocol incompatibility, complain! */
|
||||
GNUNET_break_op (0);
|
||||
return GNUNET_SYSERR;
|
||||
}
|
||||
}
|
||||
|
||||
/* check balance = total_in - total_out < withdraw-amount */
|
||||
if (GNUNET_SYSERR ==
|
||||
TALER_amount_subtract (balance,
|
||||
&total_in,
|
||||
&total_out))
|
||||
{
|
||||
/* total_in < total_out, why did the mint ever allow this!? */
|
||||
GNUNET_break_op (0);
|
||||
return GNUNET_SYSERR;
|
||||
}
|
||||
|
||||
return GNUNET_OK;
|
||||
}
|
||||
|
||||
|
||||
/**
|
||||
* Function called when we're done processing the
|
||||
* HTTP /withdraw/status request.
|
||||
@ -135,15 +325,71 @@ handle_withdraw_status_finished (void *cls,
|
||||
switch (response_code)
|
||||
{
|
||||
case MHD_HTTP_OK:
|
||||
GNUNET_break (0); // FIXME: #3773
|
||||
{
|
||||
json_t *history;
|
||||
unsigned int len;
|
||||
struct TALER_Amount balance;
|
||||
struct TALER_Amount balance_from_history;
|
||||
struct MAJ_Specification spec[] = {
|
||||
MAJ_spec_amount ("balance", &balance),
|
||||
MAJ_spec_end
|
||||
};
|
||||
|
||||
if (GNUNET_OK !=
|
||||
MAJ_parse_json (json,
|
||||
spec))
|
||||
{
|
||||
GNUNET_break_op (0);
|
||||
response_code = 0;
|
||||
break;
|
||||
}
|
||||
history = json_object_get (json,
|
||||
"history");
|
||||
if (NULL == history)
|
||||
{
|
||||
GNUNET_break_op (0);
|
||||
response_code = 0;
|
||||
break;
|
||||
}
|
||||
len = json_array_size (history);
|
||||
{
|
||||
struct TALER_MINT_ReserveHistory rhistory[len];
|
||||
|
||||
if (GNUNET_OK !=
|
||||
parse_reserve_history (history,
|
||||
&wsh->reserve_pub,
|
||||
balance.currency,
|
||||
&balance_from_history,
|
||||
len,
|
||||
rhistory))
|
||||
{
|
||||
GNUNET_break_op (0);
|
||||
response_code = 0;
|
||||
break;
|
||||
}
|
||||
if (0 !=
|
||||
TALER_amount_cmp (&balance_from_history,
|
||||
&balance))
|
||||
{
|
||||
/* mint cannot add up balances!? */
|
||||
GNUNET_break_op (0);
|
||||
response_code = 0;
|
||||
break;
|
||||
}
|
||||
wsh->cb (wsh->cb_cls,
|
||||
response_code,
|
||||
json,
|
||||
&balance,
|
||||
len,
|
||||
rhistory);
|
||||
wsh->cb = NULL;
|
||||
}
|
||||
}
|
||||
break;
|
||||
case MHD_HTTP_BAD_REQUEST:
|
||||
/* This should never happen, either us or the mint is buggy
|
||||
(or API version conflict); just pass JSON reply to the application */
|
||||
break;
|
||||
case MHD_HTTP_FORBIDDEN:
|
||||
GNUNET_break (0); // FIXME: #3773
|
||||
break;
|
||||
case MHD_HTTP_NOT_FOUND:
|
||||
/* Nothing really to verify, this should never
|
||||
happen, we should pass the JSON reply to the application */
|
||||
@ -158,9 +404,10 @@ handle_withdraw_status_finished (void *cls,
|
||||
response_code = 0;
|
||||
break;
|
||||
}
|
||||
GNUNET_break (0); // FIXME: #3773
|
||||
if (NULL != wsh->cb)
|
||||
wsh->cb (wsh->cb_cls,
|
||||
response_code,
|
||||
json,
|
||||
NULL,
|
||||
0, NULL);
|
||||
json_decref (json);
|
||||
@ -255,6 +502,7 @@ TALER_MINT_withdraw_status (struct TALER_MINT_Handle *mint,
|
||||
wsh->mint = mint;
|
||||
wsh->cb = cb;
|
||||
wsh->cb_cls = cb_cls;
|
||||
wsh->reserve_pub = *reserve_pub;
|
||||
wsh->url = MAH_path_to_url (mint,
|
||||
arg_str);
|
||||
GNUNET_free (arg_str);
|
||||
@ -455,12 +703,9 @@ withdraw_sign_payment_required (struct TALER_MINT_WithdrawSignHandle *wsh,
|
||||
{
|
||||
struct TALER_Amount balance;
|
||||
struct TALER_Amount balance_from_history;
|
||||
struct TALER_Amount total_in;
|
||||
struct TALER_Amount total_out;
|
||||
struct TALER_Amount requested_amount;
|
||||
json_t *history;
|
||||
size_t len;
|
||||
size_t off;
|
||||
struct MAJ_Specification spec[] = {
|
||||
MAJ_spec_amount ("balance", &balance),
|
||||
MAJ_spec_end
|
||||
@ -481,136 +726,25 @@ withdraw_sign_payment_required (struct TALER_MINT_WithdrawSignHandle *wsh,
|
||||
return GNUNET_SYSERR;
|
||||
}
|
||||
|
||||
/* FIXME: re-use/share this code with history processing
|
||||
on /withdraw/status above! -- #3772-#9310 */
|
||||
/* go over transaction history and compute
|
||||
total incoming and outgoing amounts */
|
||||
len = json_array_size (history);
|
||||
TALER_amount_get_zero (balance.currency,
|
||||
&total_in);
|
||||
TALER_amount_get_zero (balance.currency,
|
||||
&total_out);
|
||||
for (off=0;off<len;off++)
|
||||
{
|
||||
json_t *transaction;
|
||||
struct TALER_Amount amount;
|
||||
const char *type;
|
||||
struct MAJ_Specification hist_spec[] = {
|
||||
MAJ_spec_string ("type", &type),
|
||||
MAJ_spec_amount ("amount",
|
||||
&amount),
|
||||
/* 'wire' and 'signature' are optional depending on 'type'! */
|
||||
MAJ_spec_end
|
||||
};
|
||||
|
||||
transaction = json_array_get (history,
|
||||
off);
|
||||
if (GNUNET_OK !=
|
||||
MAJ_parse_json (transaction,
|
||||
hist_spec))
|
||||
{
|
||||
GNUNET_break_op (0);
|
||||
return GNUNET_SYSERR;
|
||||
}
|
||||
|
||||
if (0 == strcasecmp (type,
|
||||
"DEPOSIT"))
|
||||
{
|
||||
json_t *wire;
|
||||
struct TALER_MINT_ReserveHistory rhistory[len];
|
||||
|
||||
if (GNUNET_OK !=
|
||||
TALER_amount_add (&total_in,
|
||||
&total_in,
|
||||
&amount))
|
||||
parse_reserve_history (history,
|
||||
&wsh->reserve_pub,
|
||||
balance.currency,
|
||||
&balance_from_history,
|
||||
len,
|
||||
rhistory))
|
||||
{
|
||||
/* overflow in history already!? inconceivable! Bad mint! */
|
||||
GNUNET_break_op (0);
|
||||
return GNUNET_SYSERR;
|
||||
}
|
||||
wire = json_object_get (transaction,
|
||||
"wire");
|
||||
/* check 'wire' is a JSON object (no need to check wireformat,
|
||||
but we do at least expect "some" JSON object here) */
|
||||
if ( (NULL == wire) ||
|
||||
(! json_is_object (wire)) )
|
||||
{
|
||||
/* not even a JSON 'wire' specification, not acceptable */
|
||||
GNUNET_break_op (0);
|
||||
return GNUNET_SYSERR;
|
||||
}
|
||||
/* end type==DEPOSIT */
|
||||
}
|
||||
else if (0 == strcasecmp (type,
|
||||
"WITHDRAW"))
|
||||
{
|
||||
struct GNUNET_CRYPTO_EccSignaturePurpose *purpose;
|
||||
const struct TALER_WithdrawRequestPS *withdraw_purpose;
|
||||
struct TALER_Amount amount_from_purpose;
|
||||
struct MAJ_Specification withdraw_spec[] = {
|
||||
MAJ_spec_eddsa_signed_purpose ("signature",
|
||||
&purpose,
|
||||
&wsh->reserve_pub.eddsa_pub),
|
||||
MAJ_spec_end
|
||||
};
|
||||
|
||||
if (GNUNET_OK !=
|
||||
MAJ_parse_json (transaction,
|
||||
withdraw_spec))
|
||||
{
|
||||
GNUNET_break_op (0);
|
||||
return GNUNET_SYSERR;
|
||||
}
|
||||
/* Check that the signature actually signed a withdraw request */
|
||||
if ( (ntohl (purpose->purpose) != TALER_SIGNATURE_WALLET_RESERVE_WITHDRAW) ||
|
||||
(ntohl (purpose->size) != sizeof (struct TALER_WithdrawRequestPS)) )
|
||||
{
|
||||
GNUNET_break_op (0);
|
||||
MAJ_parse_free (withdraw_spec);
|
||||
return GNUNET_SYSERR;
|
||||
}
|
||||
withdraw_purpose = (const struct TALER_WithdrawRequestPS *) purpose;
|
||||
TALER_amount_ntoh (&amount_from_purpose,
|
||||
&withdraw_purpose->amount_with_fee);
|
||||
if (0 != TALER_amount_cmp (&amount,
|
||||
&amount_from_purpose))
|
||||
{
|
||||
GNUNET_break_op (0);
|
||||
MAJ_parse_free (withdraw_spec);
|
||||
return GNUNET_SYSERR;
|
||||
}
|
||||
|
||||
/* FIXME: ought to also check that the same withdraw transaction
|
||||
isn't listed twice by the mint... #3772-9310 */
|
||||
if (GNUNET_OK !=
|
||||
TALER_amount_add (&total_out,
|
||||
&total_out,
|
||||
&amount))
|
||||
{
|
||||
/* overflow in history already!? inconceivable! Bad mint! */
|
||||
GNUNET_break_op (0);
|
||||
MAJ_parse_free (withdraw_spec);
|
||||
return GNUNET_SYSERR;
|
||||
}
|
||||
/* end type==WITHDRAW */
|
||||
}
|
||||
else
|
||||
{
|
||||
/* unexpected 'type', protocol incompatibility, complain! */
|
||||
GNUNET_break_op (0);
|
||||
return GNUNET_SYSERR;
|
||||
}
|
||||
}
|
||||
|
||||
/* check balance = total_in - total_out < withdraw-amount */
|
||||
if (GNUNET_SYSERR ==
|
||||
TALER_amount_subtract (&balance_from_history,
|
||||
&total_in,
|
||||
&total_out))
|
||||
{
|
||||
/* total_in < total_out, why did the mint ever allow this!? */
|
||||
GNUNET_break_op (0);
|
||||
return GNUNET_SYSERR;
|
||||
}
|
||||
if (0 !=
|
||||
TALER_amount_cmp (&balance_from_history,
|
||||
&balance))
|
||||
|
Loading…
Reference in New Issue
Block a user