]> granicus.if.org Git - postgresql/commit
Avoid possibly-unsafe use of Windows' FormatMessage() function.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 29 Mar 2016 15:54:57 +0000 (11:54 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 29 Mar 2016 15:55:19 +0000 (11:55 -0400)
commit7abc1571652a924ba4258bda0a26df2de03b790e
treed606955e4a266c6ebf185c2ce87bf0f8f391ffdb
parent61d66c44f18c73094a50a2ef97d26cc03e171dc0
Avoid possibly-unsafe use of Windows' FormatMessage() function.

Whenever this function is used with the FORMAT_MESSAGE_FROM_SYSTEM flag,
it's good practice to include FORMAT_MESSAGE_IGNORE_INSERTS as well.
Otherwise, if the message contains any %n insertion markers, the function
will try to fetch argument strings to substitute --- which we are not
passing, possibly leading to a crash.  This is exactly analogous to the
rule about not giving printf() a format string you're not in control of.

Noted and patched by Christian Ullrich.
Back-patch to all supported branches.
src/backend/libpq/auth.c
src/backend/port/win32/socket.c
src/interfaces/libpq/fe-auth.c
src/port/dirmod.c