diff options
author | Arnd Bergmann <arnd@arndb.de> | 2019-06-18 11:21:53 +0200 |
---|---|---|
committer | Herbert Xu <herbert@gondor.apana.org.au> | 2019-06-27 14:28:01 +0800 |
commit | 149c4e6ef7788d58b9c05eed9fb85e0f5a2c3456 (patch) | |
tree | d079cb9928b3ed09a39ede7a516196897a8dfc7e /lib/test_firmware.c | |
parent | 6b5ca646ca9d99611e30f3c8f6b4837b9890eb73 (diff) | |
download | linux-149c4e6ef7788d58b9c05eed9fb85e0f5a2c3456.tar.bz2 |
crypto: testmgr - dynamically allocate crypto_shash
The largest stack object in this file is now the shash descriptor.
Since there are many other stack variables, this can push it
over the 1024 byte warning limit, in particular with clang and
KASAN:
crypto/testmgr.c:1693:12: error: stack frame size of 1312 bytes in function '__alg_test_hash' [-Werror,-Wframe-larger-than=]
Make test_hash_vs_generic_impl() do the same thing as the
corresponding eaed and skcipher functions by allocating the
descriptor dynamically. We can still do better than this,
but it brings us well below the 1024 byte limit.
Suggested-by: Eric Biggers <ebiggers@kernel.org>
Fixes: 9a8a6b3f0950 ("crypto: testmgr - fuzz hashes against their generic implementation")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Reviewed-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'lib/test_firmware.c')
0 files changed, 0 insertions, 0 deletions