drivers/net/irda/irda-usb.c: fix buffer overflow
authorJos-Vicente Gilabert <josevteg@gmail.com>
Thu, 15 Jan 2009 04:55:00 +0000 (20:55 -0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 15 Jan 2009 04:55:00 +0000 (20:55 -0800)
Taken from http://bugzilla.kernel.org/show_bug.cgi?id=12397

We're doing an sprintf of an 11-char string into an 11-char buffer.
Whoops.  It breaks firmware uploading.

Reported-by: Jos-Vicente Gilabert <josevteg@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/irda/irda-usb.c

index 29118f58a141d56aa925c8f0c353103365191a06..3a22dc41b656f02ec97281e751dd1dc0a38786fb 100644 (file)
@@ -1073,7 +1073,7 @@ static int stir421x_patch_device(struct irda_usb_cb *self)
 {
        unsigned int i;
        int ret;
-       char stir421x_fw_name[11];
+       char stir421x_fw_name[12];
        const struct firmware *fw;
        const unsigned char *fw_version_ptr; /* pointer to version string */
        unsigned long fw_version = 0;