diff options
| author | Daniel Borkmann <daniel@iogearbox.net> | 2015-11-03 11:39:20 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2015-11-03 11:29:56 -0500 |
| commit | 1d056d9c95be87725c07e514930b41c2c7174e75 (patch) | |
| tree | 74f2a040a020c31955e985b9480ed16b72fb51d7 | |
| parent | 85013ab5722268cd3c410425bd5a42feb6974a1f (diff) | |
| download | op-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.c | 2 |
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; |
