HSI: nokia-modem: Use devm_kcalloc() in nokia_modem_gpio_probe()
authorMarkus Elfring <elfring@users.sourceforge.net>
Tue, 25 Apr 2017 07:49:22 +0000 (09:49 +0200)
committerSebastian Reichel <sre@kernel.org>
Thu, 8 Jun 2017 11:21:43 +0000 (13:21 +0200)
* A multiplication for the size determination of a memory allocation
  indicated that an array data structure should be processed.
  Thus use the corresponding function "devm_kcalloc".

  This issue was detected by using the Coccinelle software.

* Replace the specification of a data structure by a pointer dereference
  to make the corresponding size determination a bit safer according to
  the Linux coding style convention.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.co.uk>
drivers/hsi/clients/nokia-modem.c

index c000780d931f80f0c767f59c6fe93d34b94a8632..f6c97a0e1fcdb0d220eed6b1db74eef7f54175e2 100644 (file)
@@ -102,8 +102,8 @@ static int nokia_modem_gpio_probe(struct device *dev)
                return -EINVAL;
        }
 
-       modem->gpios = devm_kzalloc(dev, gpio_count *
-                               sizeof(struct nokia_modem_gpio), GFP_KERNEL);
+       modem->gpios = devm_kcalloc(dev, gpio_count, sizeof(*modem->gpios),
+                                   GFP_KERNEL);
        if (!modem->gpios) {
                dev_err(dev, "Could not allocate memory for gpios\n");
                return -ENOMEM;