summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMasahiro Yamada <masahiroy@kernel.org>2023-12-03 18:49:31 +0900
committerMasahiro Yamada <masahiroy@kernel.org>2023-12-10 15:34:37 +0900
commitcc87b7c06f2a6a1fbc7e06ccf6123aada4d0b588 (patch)
tree4e182b6de1b3afbe264ac5648d0361fe912ad67b
parentcbe826b058bb3547f195144fc018957871568320 (diff)
downloadlinux-cc87b7c06f2a6a1fbc7e06ccf6123aada4d0b588.tar.gz
linux-cc87b7c06f2a6a1fbc7e06ccf6123aada4d0b588.tar.bz2
linux-cc87b7c06f2a6a1fbc7e06ccf6123aada4d0b588.zip
modpost: move __attribute__((format(printf, 2, 3))) to modpost.h
This attribute must be added to the function declaration in a header for comprehensive checking of all the callsites. Fixes: 6d9a89ea4b06 ("kbuild: declare the modpost error functions as printf like") Signed-off-by: Masahiro Yamada <masahiroy@kernel.org> Reviewed-by: Nathan Chancellor <nathan@kernel.org>
-rw-r--r--scripts/mod/modpost.c3
-rw-r--r--scripts/mod/modpost.h3
2 files changed, 3 insertions, 3 deletions
diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c
index cb6406f485a9..ca0a90158f85 100644
--- a/scripts/mod/modpost.c
+++ b/scripts/mod/modpost.c
@@ -60,8 +60,7 @@ static unsigned int nr_unresolved;
#define MODULE_NAME_LEN (64 - sizeof(Elf_Addr))
-void __attribute__((format(printf, 2, 3)))
-modpost_log(enum loglevel loglevel, const char *fmt, ...)
+void modpost_log(enum loglevel loglevel, const char *fmt, ...)
{
va_list arglist;
diff --git a/scripts/mod/modpost.h b/scripts/mod/modpost.h
index 69baf014da4f..9fe974dc1a52 100644
--- a/scripts/mod/modpost.h
+++ b/scripts/mod/modpost.h
@@ -197,7 +197,8 @@ enum loglevel {
LOG_FATAL
};
-void modpost_log(enum loglevel loglevel, const char *fmt, ...);
+void __attribute__((format(printf, 2, 3)))
+modpost_log(enum loglevel loglevel, const char *fmt, ...);
/*
* warn - show the given message, then let modpost continue running, still