Commit 0ea36374 authored by Michał Kępień's avatar Michał Kępień
Browse files

Merge branch...

Merge branch '239-don-t-use-null-as-a-argument-to-a-varargs-function-as-it-may-not-be-promoted-properly' into 'master'

Resolve "don't use NULL as a argument to a varargs function as it may not be promoted properly"

Closes #239

See merge request !228
parents 53fbf143 6aae115d
Pipeline #1596 passed with stages
in 7 minutes and 51 seconds
4944. [cleanup] Silence cppcheck portability warnings in
lib/isc/tests/buffer_test.c. [GL #239]
4943. [bug] Change 4687 consumed too much memory when running
system tests with --with-tuning=large. Reduced the
hash table size to 512 entries for 'named -m record'
......
......@@ -265,10 +265,15 @@ ATF_TC_BODY(isc_buffer_printf, tc) {
/*
* Check an empty format string is properly handled.
*
* Note: we don't use a string literal for the format string to
* avoid triggering [-Werror=format-zero-length].
* Note: we have a dummy third argument as some compilers complain
* without it.
*/
prev_used = used;
empty_fmt = "";
result = isc_buffer_printf(b, empty_fmt, NULL);
result = isc_buffer_printf(b, empty_fmt, "");
ATF_CHECK_EQ(result, ISC_R_SUCCESS);
used = isc_buffer_usedlength(b);
ATF_CHECK_EQ(prev_used, used);
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment