提交 74d9519a 编写于 作者: A Andy Polyakov

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: NRichard Levitte <levitte@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/3083)
上级 12557a34
...@@ -4,9 +4,9 @@ ...@@ -4,9 +4,9 @@
Changes between 1.1.0e and 1.1.1 [xx XXX xxxx] Changes between 1.1.0e and 1.1.1 [xx XXX xxxx]
*) Add the z modifier parsing to BIO_printf() et al formatting string, *) Add the 'z' and 'j' modifiers to BIO_printf() et al formatting string,
to be used for size_t and ssize_t (ossl_ssize_t). 'z' is to be used for [s]size_t, and 'j' - with [u]int64_t.
[Richard Levitte] [Richard Levitte, Andy Polyakov]
*) Add EC_KEY_get0_engine(), which does for EC_KEY what RSA_get0_engine() *) Add EC_KEY_get0_engine(), which does for EC_KEY what RSA_get0_engine()
does for RSA, etc. does for RSA, etc.
......
...@@ -208,6 +208,7 @@ _dopr(char **sbuffer, ...@@ -208,6 +208,7 @@ _dopr(char **sbuffer,
ch = *format++; ch = *format++;
break; break;
case 'q': case 'q':
case 'j':
cflags = DP_C_LLONG; cflags = DP_C_LLONG;
ch = *format++; ch = *format++;
break; break;
......
...@@ -173,6 +173,34 @@ static void dozutest(int test, const struct z_data_st *data, int *fail) ...@@ -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 main(int argc, char **argv)
{ {
int test = 0; int test = 0;
...@@ -246,11 +274,9 @@ int main(int argc, char **argv) ...@@ -246,11 +274,9 @@ int main(int argc, char **argv)
dozutest(test++, &zu_data[i], &fail); dozutest(test++, &zu_data[i], &fail);
} }
#if 0 for (i = 0; i < nelem(j_data); i++) {
for (i = 0; i < nelem(zi_data); i++) { dojtest(test++, &j_data[i], &fail);
dozitest(test++, &zu_data[i], &fail);
} }
#endif
#ifndef OPENSSL_NO_CRYPTO_MDEBUG #ifndef OPENSSL_NO_CRYPTO_MDEBUG
if (CRYPTO_mem_leaks_fp(stderr) <= 0) if (CRYPTO_mem_leaks_fp(stderr) <= 0)
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册