diff options
author | Andy Polyakov <appro@openssl.org> | 2017-03-29 23:10:08 +0200 |
---|---|---|
committer | Andy Polyakov <appro@openssl.org> | 2017-03-30 19:33:32 +0200 |
commit | 74d9519a68ee484db584aebc6ab6b2cb4bf98b2a (patch) | |
tree | 1e13a01e1fff525120c3029ba71f2139e687df96 /test/bioprinttest.c | |
parent | 12557a3445acc2f53321a3806f0478b998edb9a8 (diff) |
bio/b_print.c: recognize even 'j' format modifier.
'j' is specified as modifier for "greatest-width integer type", which in
practice means 64 bits on both 32- and 64-bit platforms. Since we rely
on __attribute__((__format__(__printf__,...))) to sanitize BIO_print
format, we can use it to denote [u]int64_t-s in platform-neutral manner.
Reviewed-by: Richard Levitte <levitte@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/3083)
Diffstat (limited to 'test/bioprinttest.c')
-rw-r--r-- | test/bioprinttest.c | 34 |
1 files changed, 30 insertions, 4 deletions
diff --git a/test/bioprinttest.c b/test/bioprinttest.c index c3ab6a179f..4eeb4c67a9 100644 --- a/test/bioprinttest.c +++ b/test/bioprinttest.c @@ -173,6 +173,34 @@ static void dozutest(int test, const struct z_data_st *data, int *fail) } } +struct j_data_st { + uint64_t value; + const char *format; + const char *expected; +}; +static struct j_data_st j_data[] = { + { 0xffffffffffffffffU, "%ju", "18446744073709551615" }, + { 0xffffffffffffffffU, "%jx", "ffffffffffffffff" }, + { 0x8000000000000000U, "%ju", "9223372036854775808" }, + /* + * These tests imply two's-complement, but it's the only binary + * representation we support, see test/sanitytest.c... + */ + { 0x8000000000000000U, "%ji", "-9223372036854775808" }, +}; + +static void dojtest(int test, const struct j_data_st *data, int *fail) +{ + char bio_buf[80]; + + BIO_snprintf(bio_buf, sizeof(bio_buf) - 1, data->format, data->value); + if (strcmp(bio_buf, data->expected) != 0) { + printf("Test %d failed. Expected \"%s\". Got \"%s\". " + "Format \"%s\"\n", test, data->expected, bio_buf, data->format); + *fail = 1; + } +} + int main(int argc, char **argv) { int test = 0; @@ -246,11 +274,9 @@ int main(int argc, char **argv) dozutest(test++, &zu_data[i], &fail); } -#if 0 - for (i = 0; i < nelem(zi_data); i++) { - dozitest(test++, &zu_data[i], &fail); + for (i = 0; i < nelem(j_data); i++) { + dojtest(test++, &j_data[i], &fail); } -#endif #ifndef OPENSSL_NO_CRYPTO_MDEBUG if (CRYPTO_mem_leaks_fp(stderr) <= 0) |