config.mak.dev: add -Wformat-security
commit801fa63a90e3619c91e0bb3e7a28140e6d31a097
authorJeff King <peff@peff.net>
Sat, 8 Sep 2018 16:23:31 +0000 (8 12:23 -0400)
committerJunio C Hamano <gitster@pobox.com>
Tue, 11 Sep 2018 19:50:21 +0000 (11 12:50 -0700)
tree98c4a1d31fd5862de619db804a61357709b76872
parent53f9a3e157dbbc901a02ac2c73346d375e24978c
config.mak.dev: add -Wformat-security

We currently build cleanly with -Wformat-security, and it's
a good idea to make sure we continue to do so (since calls
that trigger the warning may be security vulnerabilities).

Note that we cannot use the stronger -Wformat-nonliteral, as
there are case where we are clever with passing around
pointers to string literals. E.g., bisect_rev_setup() takes
bad_format and good_format parameters. These ultimately come
from literals, but they still trigger the warning.

Some of these might be fixable (e.g., by passing flags from
which we locally select a format), and might even be worth
fixing (not because of security, but just because it's an
easy mistake to pass the wrong format). But there are other
cases which are likely quite hard to fix (we actually
generate formats in a local buffer in some cases). So let's
punt on that for now and start with -Wformat-security, which
is supposed to catch the most important cases.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
config.mak.dev