[Qemu-devel] [PATCH] append the terminating '\0' to bootorder string

[Qemu-devel] [PATCH] append the terminating '\0' to bootorder string

[Amos Kong]

Problem was introduced in commit c8a6ae8b. The last terminating
'\0' was lost, use the right length 5 ("HALT\0").

Reported-by: Gerd Hoffmann <kraxel@redhat.com>
Signed-off-by: Amos Kong <akong@redhat.com>
---
 vl.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/vl.c b/vl.c
index aeed7f4..d3172ea 100644
--- a/vl.c
+++ b/vl.c
@@ -1273,9 +1273,9 @@ char *get_boot_devices_list(size_t *size)
 
     if (boot_strict && *size > 0) {
         list[total-1] = '\n';
-        list = g_realloc(list, total + 4);
-        memcpy(&list[total], "HALT", 4);
-        *size = total + 4;
+        list = g_realloc(list, total + 5);
+        memcpy(&list[total], "HALT", 5);
+        *size = total + 5;
     }
     return list;
 }
-- 
1.7.1

Re: [Qemu-devel] [PATCH] append the terminating '\0' to bootorder string

[Gerd Hoffmann]

On 03/20/13 11:16, Amos Kong wrote:
> Problem was introduced in commit c8a6ae8b. The last terminating
> '\0' was lost, use the right length 5 ("HALT\0").

Acked-by: Gerd Hoffmann <kraxel@redhat.com>

cheers,
  Gerd

Re: [Qemu-devel] [PATCH] append the terminating '\0' to bootorder string

[Anthony Liguori]

Applied.  Thanks.

Regards,

Anthony Liguori