summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDaniel Borkmann <daniel@iogearbox.net>2015-11-03 11:39:20 +0100
committerDavid S. Miller <davem@davemloft.net>2015-11-03 11:29:56 -0500
commit1d056d9c95be87725c07e514930b41c2c7174e75 (patch)
tree74f2a040a020c31955e985b9480ed16b72fb51d7
parent85013ab5722268cd3c410425bd5a42feb6974a1f (diff)
downloadop-kernel-dev-1d056d9c95be87725c07e514930b41c2c7174e75.zip
op-kernel-dev-1d056d9c95be87725c07e514930b41c2c7174e75.tar.gz
bpf, verifier: annotate verbose printer with __printf
The verbose() printer dumps the verifier state to user space, so let gcc take care to check calls to verbose() for (future) errors. make with W=1 correctly suggests: function might be possible candidate for 'gnu_printf' format attribute [-Wsuggest-attribute=format]. Signed-off-by: Daniel Borkmann <daniel@iogearbox.net> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--kernel/bpf/verifier.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/bpf/verifier.c b/kernel/bpf/verifier.c
index fdc88c5..c607305 100644
--- a/kernel/bpf/verifier.c
+++ b/kernel/bpf/verifier.c
@@ -214,7 +214,7 @@ static DEFINE_MUTEX(bpf_verifier_lock);
* verbose() is used to dump the verification trace to the log, so the user
* can figure out what's wrong with the program
*/
-static void verbose(const char *fmt, ...)
+static __printf(1, 2) void verbose(const char *fmt, ...)
{
va_list args;
OpenPOWER on IntegriCloud