From 38378f38d7c90c237bb8dbbe6fb4afb0c87f0c9f Mon Sep 17 00:00:00 2001 From: Christian Grothoff Date: Sat, 6 Feb 2016 19:40:12 +0100 Subject: fix compiler warning, define test wire json format --- src/wire/plugin_wire_test.c | 41 ++++++++++++++++++++++++++++++++++++----- 1 file changed, 36 insertions(+), 5 deletions(-) diff --git a/src/wire/plugin_wire_test.c b/src/wire/plugin_wire_test.c index 7940b4775..8a6f98ea2 100644 --- a/src/wire/plugin_wire_test.c +++ b/src/wire/plugin_wire_test.c @@ -226,7 +226,9 @@ test_amount_round (void *cls, /** - * Check if the given wire format JSON object is correctly formatted + * Check if the given wire format JSON object is correctly formatted. + * Right now, the only thing we require is a field + * "account_number" which must contain a positive 53-bit integer. * * @param wire the JSON wire format object * @return #GNUNET_YES if correctly formatted; #GNUNET_NO if not @@ -234,8 +236,25 @@ test_amount_round (void *cls, static int test_wire_validate (const json_t *wire) { - GNUNET_break (0); /* FIXME: we still need to define the - proper wire format for 'test' */ + json_error_t error; + json_int_t account_no; + + if (0 != + json_unpack_ex ((json_t *) wire, + &error, + 0, + "{s:I}", + "account_number", &account_no)) + { + GNUNET_break (0); + return GNUNET_SYSERR; + } + if ( (account_no < 0) || + (account_no > (1LL << 53)) ) + { + GNUNET_break (0); + return GNUNET_SYSERR; + } return GNUNET_YES; } @@ -427,7 +446,9 @@ test_execute_wire_transfer (void *cls, struct TestClosure *tc = cls; struct TALER_WIRE_ExecuteHandle *eh; json_t *wire; + json_error_t error; struct TALER_Amount amount; + json_int_t account_no; struct BufFormatP bf; if ( (buf_size <= sizeof (struct BufFormatP)) || @@ -449,16 +470,26 @@ test_execute_wire_transfer (void *cls, GNUNET_break (0); return NULL; } - GNUNET_assert (GNUNET_YES == test_wire_validate (wire)); + if (0 != + json_unpack_ex (wire, + &error, + 0, + "{s:I}", + "account_number", &account_no)) + { + GNUNET_break (0); + return NULL; + } + eh = GNUNET_new (struct TALER_WIRE_ExecuteHandle); eh->cc = cc; eh->cc_cls = cc_cls; eh->aaih = TALER_BANK_admin_add_incoming (tc->bank, &bf.wtid, &amount, - wire, + (uint64_t) account_no, &execute_cb, eh); json_decref (wire); -- cgit v1.2.3