Skip to content

Commit

Permalink
crypto: caam: - Use kmemdup() function
Browse files Browse the repository at this point in the history
Use kmemdup() rather than duplicating its implementation.

By usign kmemdup() we can also get rid of the 'val' variable.

Detected with Coccinelle script.

Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
Reviewed-by: Horia Geantă <horia.geanta@nxp.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
  • Loading branch information
Fabio Estevam authored and Herbert Xu committed Apr 20, 2018
1 parent 8a2a0dd commit b930f3a
Showing 1 changed file with 1 addition and 8 deletions.
9 changes: 1 addition & 8 deletions drivers/crypto/caam/caampkc.c
Original file line number Diff line number Diff line change
Expand Up @@ -783,19 +783,12 @@ static u8 *caam_read_rsa_crt(const u8 *ptr, size_t nbytes, size_t dstlen)
*/
static inline u8 *caam_read_raw_data(const u8 *buf, size_t *nbytes)
{
u8 *val;

caam_rsa_drop_leading_zeros(&buf, nbytes);
if (!*nbytes)
return NULL;

val = kzalloc(*nbytes, GFP_DMA | GFP_KERNEL);
if (!val)
return NULL;

memcpy(val, buf, *nbytes);

return val;
return kmemdup(buf, *nbytes, GFP_DMA | GFP_KERNEL);
}

static int caam_rsa_check_key_length(unsigned int len)
Expand Down

0 comments on commit b930f3a

Please sign in to comment.