]> granicus.if.org Git - postgresql/commit
Fix omissions in snprintf.c's coverage of standard *printf functions.
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 8 Oct 2018 23:15:55 +0000 (19:15 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 8 Oct 2018 23:15:55 +0000 (19:15 -0400)
commit7767aadd94cd252a12fa00f6122ad4dd10455791
tree45e9ef585c1a9023ddb78995b76782b4d61e282d
parent82ff0cc91d9840d1c56ca1beed58bedfde3da9a3
Fix omissions in snprintf.c's coverage of standard *printf functions.

A warning on a NetBSD box revealed to me that pg_waldump/compat.c
is using vprintf(), which snprintf.c did not provide coverage for.
This is not good if we want to have uniform *printf behavior, and
it's pretty silly to omit when it's a one-line function.

I also noted that snprintf.c has pg_vsprintf() but for some reason
it was not exposed to the outside world, creating another way in
which code might accidentally invoke the platform *printf family.

Let's just make sure that we replace all eight of the POSIX-standard
printf family.

Also, upgrade plperl.h and plpython.h to make sure that they do
their undefine/redefine rain dance for all eight, not some random
maybe-sufficient subset thereof.
src/include/port.h
src/pl/plperl/plperl.h
src/pl/plpython/plpython.h
src/port/snprintf.c