commit 4fef930af8d7fab4b6c777fa4c6e2b902359262a
author Gleb Natapov <gleb@redhat.com> Wed Feb 02 17:34:34 2011 +0200
committer Aurelien Jarno <aurelien@aurel32.net> Fri Feb 04 21:21:41 2011 +0100
tree 8119a5198bbb8ca519e62ef778e5a447f462456b
parent 72902672dc2ed6281cdb205259c1d52ecf01f6b2

do not pass NULL to strdup.

Also use qemu_strdup() instead of strdup() in bootindex code.

Signed-off-by: Gleb Natapov <gleb@redhat.com>
Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>

vl.c

diff --git a/vl.c b/vl.c
index 655617f..ed2cdfa 100644
--- a/vl.c
+++ b/vl.c
@@ -738,7 +738,7 @@
 
     node = qemu_mallocz(sizeof(FWBootEntry));
     node->bootindex = bootindex;
-    node->suffix = strdup(suffix);
+    node->suffix = suffix ? qemu_strdup(suffix) : NULL;
     node->dev = dev;
 
     QTAILQ_FOREACH(i, &fw_boot_order, link) {
@@ -785,7 +785,7 @@
         } else if (devpath) {
             bootpath = devpath;
         } else {
-            bootpath = strdup(i->suffix);
+            bootpath = qemu_strdup(i->suffix);
             assert(bootpath);
         }