net: Avoid gcc'ism in net_host_device_add
authorJan Kiszka <jan.kiszka@web.de>
Tue, 28 Apr 2009 19:25:58 +0000 (21:25 +0200)
committerAnthony Liguori <aliguori@us.ibm.com>
Fri, 1 May 2009 14:44:10 +0000 (09:44 -0500)
>> +    if (net_client_init(device, opts ? : "") < 0) {
>
> Is this a gcc extension?  Do we want to introduce this construct to the
> code base.

Valid remark, fix below.

Thanks,
Jan

-------->

Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>

net.c

diff --git a/net.c b/net.c
index 7ae1e6d..f1752d6 100644 (file)
--- a/net.c
+++ b/net.c
@@ -2085,7 +2085,7 @@ void net_host_device_add(Monitor *mon, const char *device, const char *opts)
         monitor_printf(mon, "invalid host network device %s\n", device);
         return;
     }
-    if (net_client_init(device, opts ? : "") < 0) {
+    if (net_client_init(device, opts ? opts : "") < 0) {
         monitor_printf(mon, "adding host network device %s failed\n", device);
     }
 }