summaryrefslogtreecommitdiff
path: root/Makefile.micro
diff options
context:
space:
mode:
authorDavid Mitchell <davem@iabyn.com>2013-11-28 16:46:15 +0000
committerDavid Mitchell <davem@iabyn.com>2013-11-28 17:03:49 +0000
commit5d37acd6b65eb421e938a3fde62cc1edde467dae (patch)
tree905d9b7ae0be58425baa6f089e070f0cde45e713 /Makefile.micro
parentb56aac20bc53699e4a5ea975542404fb371cf085 (diff)
downloadperl-5d37acd6b65eb421e938a3fde62cc1edde467dae.tar.gz
silence -Wformat-nonliteral compiler warnings
Due to the security risks associated with user-supplied formats being passed to C-level printf() style functions (eg %n), gcc has a -Wformat-nonliteral warning that complains whenever such a function is passed a non-literal format string. This commit silences all such warnings in core and ext/. The main changes are 1) the 'f' (format) flag in embed.fnc is now handled slightly more cleverly. Rather than just applying to functions whose last arg is '...' (and where the format arg is assumed to be the previous arg), it can now handle non-'...' functions: arg checking is disabled, but format checking is sill done: it works by assuming that an arg called 'fmt', 'pat' or 'f' is the format string (and dies if fails to find exactly one such arg). 2) with the new embed.fnc functionally, more functions have been marked with the 'f' flag. When such a function passes its fmt arg onto an inner printf-like function, we simply disable the warning for that call using GCC_DIAG_IGNORE(-Wformat-nonliteral), since we know that the caller must have already checked it. 3) In quite a few places the format string isn't literal, but it *is* constant (e.g. PL_warn_uninit_sv). For those cases, again disable the warning. 4) In pp_formline(), a particular format was was one of several different literal strings depending on circumstances. Rather than assigning this string to a temporary variable, incorporate the ?: branches directly in the function call arg. gcc is clever enough to decide the arg is then always literal.
Diffstat (limited to 'Makefile.micro')
0 files changed, 0 insertions, 0 deletions