disable /keys cherrypicking until #5315 is addressed

This commit is contained in:
Florian Dold 2018-04-06 00:58:15 +02:00
parent cb1548d1a0
commit 1b3bf1a001
No known key found for this signature in database
GPG Key ID: D2E4F00F29D02A4B
4 changed files with 19 additions and 3 deletions

View File

@ -867,7 +867,7 @@ keys_completed_cb (void *cls,
{ {
/* offsets will map 1:1 */ /* offsets will map 1:1 */
unsigned int off = aold->denom_keys[j] - kd_old.denom_keys; unsigned int off = aold->denom_keys[j] - kd_old.denom_keys;
/* FIXME(#5315): this should not be an assert! */
GNUNET_assert (off < kd_old.num_denom_keys); GNUNET_assert (off < kd_old.num_denom_keys);
anew->denom_keys[j] = &kd.denom_keys[off]; anew->denom_keys[j] = &kd.denom_keys[off];
} }
@ -1127,7 +1127,7 @@ request_keys (struct TALER_EXCHANGE_Handle *exchange)
GNUNET_assert (NULL == exchange->kr); GNUNET_assert (NULL == exchange->kr);
kr = GNUNET_new (struct KeysRequest); kr = GNUNET_new (struct KeysRequest);
kr->exchange = exchange; kr->exchange = exchange;
if (GNUNET_YES == MAH_handle_is_ready (exchange)) if (GNUNET_YES == MAH_handle_is_ready (exchange) && !TALER_EXCHANGE_API_DISABLE_CHERRYPICKING)
{ {
char *arg; char *arg;

View File

@ -579,7 +579,11 @@ run (void *cls)
/* Re-download and check /keys */ /* Re-download and check /keys */
{ .oc = OC_CHECK_KEYS, { .oc = OC_CHECK_KEYS,
.details.check_keys.generation = 2, .details.check_keys.generation = 2,
#if TALER_EXCHANGE_API_DISABLE_CHERRYPICKING
.details.check_keys.num_denom_keys = 12
#else
.details.check_keys.num_denom_keys = 8 .details.check_keys.num_denom_keys = 8
#endif
}, },
{ .oc = OC_END } { .oc = OC_END }
}; };

View File

@ -84,7 +84,12 @@ run (void *cls,
SIGUSR1), SIGUSR1),
TALER_TESTING_cmd_check_keys ("check-keys-2", TALER_TESTING_cmd_check_keys ("check-keys-2",
2, 8, 2,
#if TALER_EXCHANGE_API_DISABLE_CHERRYPICKING
12,
#else
8,
#endif
is->exchange), is->exchange),

View File

@ -31,6 +31,13 @@
/* ********************* /keys *********************** */ /* ********************* /keys *********************** */
/**
* Temporarily disable cherrypicking until it's fixed.
* See #5315.
*/
#define TALER_EXCHANGE_API_DISABLE_CHERRYPICKING 1
/** /**
* List of possible options to be passed to * List of possible options to be passed to
* #TALER_EXCHANGE_connect(). * #TALER_EXCHANGE_connect().