Commit 1d056d9c authored by Daniel Borkmann's avatar Daniel Borkmann Committed by David S. Miller

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: default avatarDaniel Borkmann <daniel@iogearbox.net>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 85013ab5
...@@ -214,7 +214,7 @@ static DEFINE_MUTEX(bpf_verifier_lock); ...@@ -214,7 +214,7 @@ static DEFINE_MUTEX(bpf_verifier_lock);
* verbose() is used to dump the verification trace to the log, so the user * verbose() is used to dump the verification trace to the log, so the user
* can figure out what's wrong with the program * 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; va_list args;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment