usb: gadget: renesas_usbhs: Use kcalloc instead of kzalloc to allocate array
authorThomas Meyer <thomas@m3y3r.de>
Tue, 29 Nov 2011 21:08:00 +0000 (22:08 +0100)
committerGreg Kroah-Hartman <gregkh@suse.de>
Sat, 10 Dec 2011 00:18:19 +0000 (16:18 -0800)
The advantage of kcalloc is, that will prevent integer overflows which could
result from the multiplication of number of elements and size and it is also
a bit nicer to read.

The semantic patch that makes this change is available
in https://lkml.org/lkml/2011/11/25/107

Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/usb/renesas_usbhs/mod_host.c

index bade761a1e52b783a63dfb462f8b11ef4c111999..75659e0c735d3dc787f3bfb6d7e1aaaa96996af4 100644 (file)
@@ -1268,7 +1268,7 @@ int usbhs_mod_host_probe(struct usbhs_priv *priv)
                return -ENOMEM;
        }
 
-       pipe_info = kzalloc(sizeof(*pipe_info) * pipe_size, GFP_KERNEL);
+       pipe_info = kcalloc(pipe_size, sizeof(*pipe_info), GFP_KERNEL);
        if (!pipe_info) {
                dev_err(dev, "Could not allocate pipe_info\n");
                goto usbhs_mod_host_probe_err;