diff options
| author | Thomas Hindoe Paaboel Andersen <phomes@gmail.com> | 2014-01-09 16:42:19 -0500 |
|---|---|---|
| committer | Anthony G. Basile <blueness@gentoo.org> | 2014-01-09 16:42:19 -0500 |
| commit | 079a9c8229692c8c68899a9ab06693f830c1a091 (patch) | |
| tree | bd5a431d75c56e736e0e7c638ecbfb22d115ad94 /src/udev | |
| parent | 46029d273782b441895b23bf5720ebe23d42351f (diff) | |
Add more _printf_'s for format-nonliterals
Clang is a bit more strict wrt format-nonliterals:
http://clang.llvm.org/docs/LanguageExtensions.html#format-string-checking
Adding these extra printf attributes also makes gcc able to find more
problems. E.g. this patch uncovers a format issue in udev-builtin-path_id.c
Some parts looked intetional about breaking the format-nonliteral check.
I added some supression for warnings there.
Signed-off-by: Anthony G. Basile <blueness@gentoo.org>
Diffstat (limited to 'src/udev')
| -rw-r--r-- | src/udev/udev-builtin-kmod.c | 1 | ||||
| -rw-r--r-- | src/udev/udev-builtin-path_id.c | 1 |
2 files changed, 2 insertions, 0 deletions
diff --git a/src/udev/udev-builtin-kmod.c b/src/udev/udev-builtin-kmod.c index 6fdd8aac4e..efd3e3e3cb 100644 --- a/src/udev/udev-builtin-kmod.c +++ b/src/udev/udev-builtin-kmod.c @@ -73,6 +73,7 @@ static int load_module(struct udev *udev, const char *alias) #endif } +_printf_(6,0) static void udev_kmod_log(void *data, int priority, const char *file, int line, const char *fn, const char *format, va_list args) { diff --git a/src/udev/udev-builtin-path_id.c b/src/udev/udev-builtin-path_id.c index da0273197b..f85b091763 100644 --- a/src/udev/udev-builtin-path_id.c +++ b/src/udev/udev-builtin-path_id.c @@ -32,6 +32,7 @@ #include "udev.h" +_printf_(2,3) static int path_prepend(char **path, const char *fmt, ...) { va_list va; |