[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 06/14] acpi-build: Fix compiler warning (missing gnu_
From: |
Michael Tokarev |
Subject: |
[Qemu-devel] [PULL 06/14] acpi-build: Fix compiler warning (missing gnu_printf format attribute) |
Date: |
Fri, 6 Dec 2013 23:15:33 +0400 |
From: Stefan Weil <address@hidden>
gcc 4.8.2 reports this warning when extra warnings are enabled (-Wextra):
CC m68k-softmmu/hw/m68k/mcf5206.o
hw/i386/acpi-build.c: In function ‘build_append_nameseg’:
hw/i386/acpi-build.c:294:5: error:
function might be possible candidate for ‘gnu_printf’ format attribute
[-Werror=suggest-attribute=format]
g_string_vprintf(s, format, args);
^
When this warning is fixed, there is a new compiler warning:
CC i386-softmmu/hw/i386/acpi-build.o
hw/i386/acpi-build.c: In function ‘build_append_notify’:
hw/i386/acpi-build.c:632:5: error:
format not a string literal and no format arguments [-Werror=format-security]
build_append_nameseg(method, name);
^
This is fixed here, too.
Signed-off-by: Stefan Weil <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>
---
hw/i386/acpi-build.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
index 1f22fb6..befc39f 100644
--- a/hw/i386/acpi-build.c
+++ b/hw/i386/acpi-build.c
@@ -285,7 +285,8 @@ static inline void build_append_array(GArray *array, GArray
*val)
g_array_append_vals(array, val->data, val->len);
}
-static void build_append_nameseg(GArray *array, const char *format, ...)
+static void GCC_FMT_ATTR(2, 3)
+build_append_nameseg(GArray *array, const char *format, ...)
{
/* It would be nicer to use g_string_vprintf but it's only there in 2.22 */
char s[] = "XXXX";
@@ -630,7 +631,7 @@ build_append_notify(GArray *device, const char *name,
GArray *method = build_alloc_array();
uint8_t op = 0x14; /* MethodOp */
- build_append_nameseg(method, name);
+ build_append_nameseg(method, "%s", name);
build_append_byte(method, 0x02); /* MethodFlags: ArgCount */
for (i = skip; i < count; i++) {
GArray *target = build_alloc_array();
--
1.7.10.4
- [Qemu-devel] [PULL 05/14] .gitignore: Ignore config.status, (continued)
- [Qemu-devel] [PULL 05/14] .gitignore: Ignore config.status, Michael Tokarev, 2013/12/06
- [Qemu-devel] [PULL 04/14] gtk: Replace conditional debug messages by trace methods, Michael Tokarev, 2013/12/06
- [Qemu-devel] [PULL 01/14] trace: Remove trace.h from console.h (less dependencies), Michael Tokarev, 2013/12/06
- [Qemu-devel] [PULL 12/14] Don't crash on keyboard input with no handler, Michael Tokarev, 2013/12/06
- [Qemu-devel] [PULL 10/14] libcacard/cac: Remove unused statement (value stored is never read), Michael Tokarev, 2013/12/06
- [Qemu-devel] [PULL 14/14] eeprom93xx: fix coding style, Michael Tokarev, 2013/12/06
- [Qemu-devel] [PULL 08/14] misc: Replace 'struct QEMUTimer' by 'QEMUTimer', Michael Tokarev, 2013/12/06
- [Qemu-devel] [PULL 11/14] libcacard/vcard_emul_nss: Remove unused statement (value stored is never read), Michael Tokarev, 2013/12/06
- [Qemu-devel] [PULL 13/14] hw/ppc/mac.h: remove unused BIOS_FILENAME definition, Michael Tokarev, 2013/12/06
- [Qemu-devel] [PULL 07/14] qobject: Fix compiler warning (missing gnu_printf format attribute), Michael Tokarev, 2013/12/06
- [Qemu-devel] [PULL 06/14] acpi-build: Fix compiler warning (missing gnu_printf format attribute),
Michael Tokarev <=
- [Qemu-devel] [PULL 09/14] virtio-net: fix the indent, Michael Tokarev, 2013/12/06