Patchwork [v2] Avoid crash on '-usbdevice <device>' without parameters

login
register
mail settings
Submitter Jan Kiszka
Date March 7, 2010, 11:17 a.m.
Message ID <4B938B34.60102@web.de>
Download mbox | patch
Permalink /patch/47077/
State New
Headers show

Comments

Jan Kiszka - March 7, 2010, 11:17 a.m.
Many usbdevice_init implementors assume params is non-NULL.

Signed-off-by: Jan Kiszka <jan.kiszka@web.de>
---

Build fixed - sorry.

 hw/usb-bus.c |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

Patch

diff --git a/hw/usb-bus.c b/hw/usb-bus.c
index 7c82314..3bb8986 100644
--- a/hw/usb-bus.c
+++ b/hw/usb-bus.c
@@ -264,7 +264,8 @@  USBDevice *usbdevice_create(const char *cmdline)
     USBBus *bus = usb_bus_find(-1 /* any */);
     DeviceInfo *info;
     USBDeviceInfo *usb;
-    char driver[32], *params;
+    char driver[32];
+    const char *params;
     int len;

     params = strchr(cmdline,':');
@@ -275,6 +276,7 @@  USBDevice *usbdevice_create(const char *cmdline)
             len = sizeof(driver);
         pstrcpy(driver, len, cmdline);
     } else {
+        params = "";
         pstrcpy(driver, sizeof(driver), cmdline);
     }