提交 be62a2eb 编写于 作者: M Markus Armbruster 提交者: Stefan Hajnoczi

Strip trailing '\n' from error_report()'s first argument (again)

Commit 6daf194d got rid of them, but Hans and Gerd added some more
lately.  Tracked down with this Coccinelle semantic patch:

@r@
    expression fmt;
    position p;
@@
    error_report(fmt, ...)@p
@script:python@
    fmt << r.fmt;
    p << r.p;
@@
if "\\n" in str(fmt):
    print "%s:%s:%s:%s" % (p[0].file, p[0].line, p[0].column, fmt)
Signed-off-by: NMarkus Armbruster <armbru@redhat.com>
上级 d70d6b31
...@@ -137,7 +137,7 @@ USBDevice *usb_create(USBBus *bus, const char *name) ...@@ -137,7 +137,7 @@ USBDevice *usb_create(USBBus *bus, const char *name)
bus = usb_bus_find(-1); bus = usb_bus_find(-1);
if (!bus) if (!bus)
return NULL; return NULL;
error_report("%s: no bus specified, using \"%s\" for \"%s\"\n", error_report("%s: no bus specified, using \"%s\" for \"%s\"",
__FUNCTION__, bus->qbus.name, name); __FUNCTION__, bus->qbus.name, name);
} }
#endif #endif
...@@ -152,12 +152,12 @@ USBDevice *usb_create_simple(USBBus *bus, const char *name) ...@@ -152,12 +152,12 @@ USBDevice *usb_create_simple(USBBus *bus, const char *name)
int rc; int rc;
if (!dev) { if (!dev) {
error_report("Failed to create USB device '%s'\n", name); error_report("Failed to create USB device '%s'", name);
return NULL; return NULL;
} }
rc = qdev_init(&dev->qdev); rc = qdev_init(&dev->qdev);
if (rc < 0) { if (rc < 0) {
error_report("Failed to initialize USB device '%s'\n", name); error_report("Failed to initialize USB device '%s'", name);
return NULL; return NULL;
} }
return dev; return dev;
...@@ -244,7 +244,7 @@ int usb_claim_port(USBDevice *dev) ...@@ -244,7 +244,7 @@ int usb_claim_port(USBDevice *dev)
} }
} }
if (port == NULL) { if (port == NULL) {
error_report("Error: usb port %s (bus %s) not found (in use?)\n", error_report("Error: usb port %s (bus %s) not found (in use?)",
dev->port_path, bus->qbus.name); dev->port_path, bus->qbus.name);
return -1; return -1;
} }
...@@ -255,7 +255,7 @@ int usb_claim_port(USBDevice *dev) ...@@ -255,7 +255,7 @@ int usb_claim_port(USBDevice *dev)
} }
if (bus->nfree == 0) { if (bus->nfree == 0) {
error_report("Error: tried to attach usb device %s to a bus " error_report("Error: tried to attach usb device %s to a bus "
"with no free ports\n", dev->product_desc); "with no free ports", dev->product_desc);
return -1; return -1;
} }
port = QTAILQ_FIRST(&bus->free); port = QTAILQ_FIRST(&bus->free);
...@@ -302,7 +302,7 @@ int usb_device_attach(USBDevice *dev) ...@@ -302,7 +302,7 @@ int usb_device_attach(USBDevice *dev)
if (!(port->speedmask & dev->speedmask)) { if (!(port->speedmask & dev->speedmask)) {
error_report("Warning: speed mismatch trying to attach " error_report("Warning: speed mismatch trying to attach "
"usb device %s to bus %s\n", "usb device %s to bus %s",
dev->product_desc, bus->qbus.name); dev->product_desc, bus->qbus.name);
return -1; return -1;
} }
......
...@@ -174,7 +174,7 @@ static void usbredir_log(void *priv, int level, const char *msg) ...@@ -174,7 +174,7 @@ static void usbredir_log(void *priv, int level, const char *msg)
return; return;
} }
error_report("%s\n", msg); error_report("%s", msg);
} }
static void usbredir_log_data(USBRedirDevice *dev, const char *desc, static void usbredir_log_data(USBRedirDevice *dev, const char *desc,
...@@ -193,7 +193,7 @@ static void usbredir_log_data(USBRedirDevice *dev, const char *desc, ...@@ -193,7 +193,7 @@ static void usbredir_log_data(USBRedirDevice *dev, const char *desc,
for (j = 0; j < 8 && i + j < len; j++) { for (j = 0; j < 8 && i + j < len; j++) {
n += sprintf(buf + n, " %02X", data[i + j]); n += sprintf(buf + n, " %02X", data[i + j]);
} }
error_report("%s\n", buf); error_report("%s", buf);
} }
} }
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册