summaryrefslogtreecommitdiffstats
path: root/drivers/crypto/caam
diff options
context:
space:
mode:
authorMarek Vasut <marex@denx.de>2014-04-24 20:05:20 +0200
committerHerbert Xu <herbert@gondor.apana.org.au>2014-05-08 21:58:10 +0800
commite22cdcfa13d9799ea46245946ae09b4a328ff27d (patch)
treeda2fad6781a9c741c55230d2cfd732130001d928 /drivers/crypto/caam
parent4f0fa52a5d89d763423cb9b428e988ea824fd95a (diff)
downloadop-kernel-dev-e22cdcfa13d9799ea46245946ae09b4a328ff27d.zip
op-kernel-dev-e22cdcfa13d9799ea46245946ae09b4a328ff27d.tar.gz
crypto: caam - Kill SPRINTFCAT() with fire
This macro is just like an encyclopedia of string handling done wrong. This must die. This is so wrong on so many levels. Signed-off-by: Marek Vasut <marex@denx.de> Cc: Herbert Xu <herbert@gondor.apana.org.au> Cc: Horia Geanta <horia.geanta@freescale.com> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'drivers/crypto/caam')
-rw-r--r--drivers/crypto/caam/error.c10
1 files changed, 0 insertions, 10 deletions
diff --git a/drivers/crypto/caam/error.c b/drivers/crypto/caam/error.c
index 66a7461..7ed2e4f 100644
--- a/drivers/crypto/caam/error.c
+++ b/drivers/crypto/caam/error.c
@@ -158,16 +158,6 @@ static const char * const rng_err_id_list[] = {
"Secure key generation",
};
-#define SPRINTFCAT(str, format, param, max_alloc) \
-{ \
- char *tmp; \
- \
- tmp = kmalloc(sizeof(format) + max_alloc, GFP_ATOMIC); \
- sprintf(tmp, format, param); \
- strcat(str, tmp); \
- kfree(tmp); \
-}
-
static void report_ccb_status(struct device *jrdev, u32 status,
const char *error, char *__outstr)
{
OpenPOWER on IntegriCloud