Skip to contentSkip to conversation

acpi: specify format for build_append_namestring

PATCH In review Michael S. Tsirkin 2 participants 2 messages Mar 11, 2015

lore
Share & tools
lore

Michael S. Tsirkin

author
#0 lore raw

PATCH 1/1 acpi: specify format for build_append_namestring

Will catch users if we misused it.

Signed-off-by Michael S. Tsirkin <[email protected]>
---
 hw/acpi/aml-build.c | 1 +
 1 file changed, 1 insertion(+)
hw/acpi/aml-build.c +1 -0
index ace180b..6242908 100644
@@ -141,6 +141,7 @@ build_append_namestringv(GArray *array, const char *format, va_list ap)
     g_strfreev(segs);
 }
 
+GCC_FMT_ATTR(2, 3)
 static void build_append_namestring(GArray *array, const char *format, ...)
 {
     va_list ap;
Signature · 1 line
MST
Reviewed-by on v1 1/1
#1 lore raw
Quoted 3 lines · On Wed, 11 Mar 2015 07:55:20 +0100 "Michael S. Tsirkin" <[email protected]> wrote:

Will catch users if we misused it.

Signed-off-by Michael S. Tsirkin <[email protected]>
Reviewed-by Igor Mammedov <[email protected]>
Quoted 16 lines
---
 hw/acpi/aml-build.c | 1 +
 1 file changed, 1 insertion(+)
hw/acpi/aml-build.c +1 -0
index ace180b..6242908 100644
@@ -141,6 +141,7 @@ build_append_namestringv(GArray *array, const char *format, va_list ap)
     g_strfreev(segs);
 }
 
+GCC_FMT_ATTR(2, 3)
 static void build_append_namestring(GArray *array, const char *format, ...)
 {
     va_list ap;