qemu-option: Fix uninitialized value in append_option_parameter

When dest is NULL, i.e. a new copy of the list is created, we don't get a
properly terminated list after the realloc. Initialize it as an empty list.

Signed-off-by Kevin Wolf <[email protected]>

---

Xudong, can you please try this one? I think it should fix your qemu-img
problem.

 qemu-option.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

qemu-option.c +1 -0

index acd74f9..f884865 100644
@@ -378,6 +378,7 @@ QEMUOptionParameter
     *append_option_parameters(QEMUOptionParameter *dest, num_options
+= count_option_parameters(list); 
 
     dest = qemu_realloc(dest, (num_options + 1) *
 sizeof(QEMUOptionParameter)); +    dest[num_dest_options].name = NULL;
 
     while (list && list->name) {

         if (get_option_parameter(dest, list->name) == NULL) {

When dest is NULL, i.e. a new copy of the list is created, we don't get a properly terminated list after the realloc. Initialize it as an empty list.

Signed-off-by Kevin Wolf<[email protected]>

---

Xudong, can you please try this one? I think it should fix your qemu-img problem.

  qemu-option.c |    1 +
  1 files changed, 1 insertions(+), 0 deletions(-)

qemu-option.c +1 -0

index acd74f9..f884865 100644
@@ -378,6 +378,7 @@ QEMUOptionParameter *append_option_parameters(QEMUOptionParameter *dest,
      num_options += count_option_parameters(list);
 
      dest = qemu_realloc(dest, (num_options + 1) * sizeof(QEMUOptionParameter));
+    dest[num_dest_options].name = NULL;
 
      while (list&&  list->name) {
          if (get_option_parameter(dest, list->name) == NULL) {