[qemu-xen master] usb: Fix build with newer gcc

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[qemu-xen master] usb: Fix build with newer gcc

patchbot
commit 61b4e0616e77b5fe715f70e4d37f4b3aaef1998d
Author:     Eric Blake <[hidden email]>
AuthorDate: Mon Jul 17 10:13:34 2017 -0500
Commit:     Anthony PERARD <[hidden email]>
CommitDate: Wed Aug 2 16:25:14 2017 +0100

    usb: Fix build with newer gcc
   
    gcc 7 is pickier about our sources:
   
    hw/usb/bus.c: In function ‘usb_port_location’:
    hw/usb/bus.c:410:66: error: ‘%d’ directive output may be truncated writing between 1 and 11 bytes into a region of size between 0 and 15 [-Werror=format-truncation=]
             snprintf(downstream->path, sizeof(downstream->path), "%s.%d",
                                                                      ^~
    hw/usb/bus.c:410:9: note: ‘snprintf’ output between 3 and 28 bytes into a destination of size 16
             snprintf(downstream->path, sizeof(downstream->path), "%s.%d",
             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                      upstream->path, portnr);
                      ~~~~~~~~~~~~~~~~~~~~~~~
   
    But we know that there are at most 5 levels of USB hubs, with at
    most two digits per level; that plus the separating dots means we
    use at most 15 bytes (including trailing NUL) of our 16-byte field.
    Adding an assertion to show gcc that we checked for truncation is
    enough to shut up the false-positive warning.
   
    Inspired by an idea by Dr. David Alan Gilbert <[hidden email]>.
   
    Signed-off-by: Eric Blake <[hidden email]>
    Reviewed-by: Philippe Mathieu-Daudé <[hidden email]>
    Message-id: [hidden email]
    Signed-off-by: Gerd Hoffmann <[hidden email]>
    (cherry picked from commit 121829cb2160e9cd82482c1542699fa589688106)
---
 hw/usb/bus.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/hw/usb/bus.c b/hw/usb/bus.c
index 24f1608..cf93f4d 100644
--- a/hw/usb/bus.c
+++ b/hw/usb/bus.c
@@ -407,8 +407,10 @@ void usb_register_companion(const char *masterbus, USBPort *ports[],
 void usb_port_location(USBPort *downstream, USBPort *upstream, int portnr)
 {
     if (upstream) {
-        snprintf(downstream->path, sizeof(downstream->path), "%s.%d",
-                 upstream->path, portnr);
+        int l = snprintf(downstream->path, sizeof(downstream->path), "%s.%d",
+                         upstream->path, portnr);
+        /* Max string is nn.nn.nn.nn.nn, which fits in 16 bytes */
+        assert(l < sizeof(downstream->path));
         downstream->hubcount = upstream->hubcount + 1;
     } else {
         snprintf(downstream->path, sizeof(downstream->path), "%d", portnr);
--
generated by git-patchbot for /home/xen/git/qemu-xen.git#master


_______________________________________________
Xen-changelog mailing list
[hidden email]
https://lists.xenproject.org/xen-changelog
Loading...