crypto: caam - Use kmalloc_array() in ahash_setkey()
authorMarkus Elfring <elfring@users.sourceforge.net>
Thu, 15 Sep 2016 09:20:09 +0000 (11:20 +0200)
committerHerbert Xu <herbert@gondor.apana.org.au>
Thu, 22 Sep 2016 10:27:39 +0000 (18:27 +0800)
* A multiplication for the size determination of a memory allocation
  indicated that an array data structure should be processed.
  Thus use the corresponding function "kmalloc_array".

  This issue was detected by using the Coccinelle software.

* Replace the specification of a data type 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: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/caam/caamhash.c

index 9d7fc9e..f19df8f 100644 (file)
@@ -525,8 +525,9 @@ static int ahash_setkey(struct crypto_ahash *ahash,
 #endif
 
        if (keylen > blocksize) {
-               hashed_key = kmalloc(sizeof(u8) * digestsize, GFP_KERNEL |
-                                    GFP_DMA);
+               hashed_key = kmalloc_array(digestsize,
+                                          sizeof(*hashed_key),
+                                          GFP_KERNEL | GFP_DMA);
                if (!hashed_key)
                        return -ENOMEM;
                ret = hash_digest_key(ctx, key, &keylen, hashed_key,