diff options
author | Marek Vasut <marex@denx.de> | 2014-04-24 18:05:20 (GMT) |
---|---|---|
committer | Matthew Weigel <Matthew.Weigel@freescale.com> | 2014-12-11 18:41:27 (GMT) |
commit | 90f84825dc383996b204297a318e2cf6c73909b0 (patch) | |
tree | ce6cc5957e859afbff70e4d38c425f1eb0a62fea | |
parent | a50dcd896041fae96e32480e8589deaba30bbc2a (diff) | |
download | linux-fsl-qoriq-90f84825dc383996b204297a318e2cf6c73909b0.tar.xz |
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>
(cherry picked from commit e22cdcfa13d9799ea46245946ae09b4a328ff27d)
-rw-r--r-- | drivers/crypto/caam/error.c | 10 |
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) { |