]> git.proxmox.com Git - qemu.git/commitdiff
acpi-build: fix build on glib < 2.22
authorMichael S. Tsirkin <mst@redhat.com>
Thu, 21 Nov 2013 09:19:58 +0000 (11:19 +0200)
committerMichael S. Tsirkin <mst@redhat.com>
Thu, 21 Nov 2013 14:28:21 +0000 (16:28 +0200)
g_string_vprintf was only introduced in 2.24 so switch to vsnprintf
instead.  A bit uglier but name size is fixed at 4 bytes here so it's
easy.

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Reported-by: Richard Henderson <rth@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
hw/i386/acpi-build.c

index 486e7055a6e8fd8424e233f7b0445d49fa916455..59a17dfbd37b7ed9cacb1a97b5fae14dcf7605aa 100644 (file)
@@ -287,16 +287,17 @@ static inline void build_append_array(GArray *array, GArray *val)
 
 static void build_append_nameseg(GArray *array, const char *format, ...)
 {
-    GString *s = g_string_new("");
+    /* It would be nicer to use g_string_vprintf but it's only there in 2.22 */
+    char s[] = "XXXX";
+    int len;
     va_list args;
 
     va_start(args, format);
-    g_string_vprintf(s, format, args);
+    len = vsnprintf(s, sizeof s, format, args);
     va_end(args);
 
-    assert(s->len == 4);
-    g_array_append_vals(array, s->str, s->len);
-    g_string_free(s, true);
+    assert(len == 4);
+    g_array_append_vals(array, s, len);
 }
 
 /* 5.4 Definition Block Encoding */