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: Markus Armbruster <armbru@redhat.com>
			
			
This commit is contained in:
		
							parent
							
								
									d70d6b3109
								
							
						
					
					
						commit
						be62a2ebab
					
				
							
								
								
									
										12
									
								
								hw/usb-bus.c
								
								
								
								
							
							
						
						
									
										12
									
								
								hw/usb-bus.c
								
								
								
								
							| 
						 | 
				
			
			@ -137,7 +137,7 @@ USBDevice *usb_create(USBBus *bus, const char *name)
 | 
			
		|||
        bus = usb_bus_find(-1);
 | 
			
		||||
        if (!bus)
 | 
			
		||||
            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);
 | 
			
		||||
    }
 | 
			
		||||
#endif
 | 
			
		||||
| 
						 | 
				
			
			@ -152,12 +152,12 @@ USBDevice *usb_create_simple(USBBus *bus, const char *name)
 | 
			
		|||
    int rc;
 | 
			
		||||
 | 
			
		||||
    if (!dev) {
 | 
			
		||||
        error_report("Failed to create USB device '%s'\n", name);
 | 
			
		||||
        error_report("Failed to create USB device '%s'", name);
 | 
			
		||||
        return NULL;
 | 
			
		||||
    }
 | 
			
		||||
    rc = qdev_init(&dev->qdev);
 | 
			
		||||
    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 dev;
 | 
			
		||||
| 
						 | 
				
			
			@ -244,7 +244,7 @@ int usb_claim_port(USBDevice *dev)
 | 
			
		|||
            }
 | 
			
		||||
        }
 | 
			
		||||
        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);
 | 
			
		||||
            return -1;
 | 
			
		||||
        }
 | 
			
		||||
| 
						 | 
				
			
			@ -255,7 +255,7 @@ int usb_claim_port(USBDevice *dev)
 | 
			
		|||
        }
 | 
			
		||||
        if (bus->nfree == 0) {
 | 
			
		||||
            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;
 | 
			
		||||
        }
 | 
			
		||||
        port = QTAILQ_FIRST(&bus->free);
 | 
			
		||||
| 
						 | 
				
			
			@ -302,7 +302,7 @@ int usb_device_attach(USBDevice *dev)
 | 
			
		|||
 | 
			
		||||
    if (!(port->speedmask & dev->speedmask)) {
 | 
			
		||||
        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);
 | 
			
		||||
        return -1;
 | 
			
		||||
    }
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -174,7 +174,7 @@ static void usbredir_log(void *priv, int level, const char *msg)
 | 
			
		|||
        return;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    error_report("%s\n", msg);
 | 
			
		||||
    error_report("%s", msg);
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
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++) {
 | 
			
		||||
            n += sprintf(buf + n, " %02X", data[i + j]);
 | 
			
		||||
        }
 | 
			
		||||
        error_report("%s\n", buf);
 | 
			
		||||
        error_report("%s", buf);
 | 
			
		||||
    }
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in New Issue