diff options
author | Dmitry Vyukov <dvyukov@google.com> | 2021-08-12 19:37:29 +0200 |
---|---|---|
committer | Dmitry Vyukov <dvyukov@google.com> | 2021-08-13 13:43:57 +0200 |
commit | 9c8f888f5fcafe8a9b69e7d652dc7328612f8ec6 (patch) | |
tree | cb1b8d222e7320c5a0eb5f274c558db249268cc7 /clang/lib/Frontend/CompilerInvocation.cpp | |
parent | e772e25547b748812d61561a8a444e3c42a4cfc2 (diff) | |
download | llvm-9c8f888f5fcafe8a9b69e7d652dc7328612f8ec6.zip llvm-9c8f888f5fcafe8a9b69e7d652dc7328612f8ec6.tar.gz llvm-9c8f888f5fcafe8a9b69e7d652dc7328612f8ec6.tar.bz2 |
sanitizer_common: prepare for enabling format string checking
The __attribute__((format)) was added somewhere in 2012,
the lost during refactoring, then re-added in 2014 but
to te source files, which is a no-op.
Move it back to header files so that it actually takes effect.
But over the past 7 years we've accumulated whole lot of
format string bugs of different types, so disable the warning
with -Wno-format for now for incremental clean up.
Among the bugs that it warns about are all kinds of bad things:
- wrong sizes of arguments
- missing/excessive arguments
- printing wrong things (e.g. *ptr instead of ptr)
- completely messed up format strings
- security issues where external string is used as format
Reviewed By: vitalybuka
Differential Revision: https://reviews.llvm.org/D107977
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions