diff options
author | Christoph Manszewski <c.manszewski@samsung.com> | 2018-09-17 18:09:27 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-11-20 20:47:43 +0300 |
commit | ef089d9b58a79eaa5052d50e5becc7549de81dee (patch) | |
tree | 5836620785fd37dc7dafb63f05439999d1b31c60 /crypto | |
parent | d34465e722e7f2d753652ff03d94bd4a780c4509 (diff) | |
download | linux-ef089d9b58a79eaa5052d50e5becc7549de81dee.tar.xz |
crypto: s5p-sss: Fix race in error handling
[ Upstream commit 5842cd44786055231b233ed5ed98cdb63ffb7db3 ]
Remove a race condition introduced by error path in functions:
s5p_aes_interrupt and s5p_aes_crypt_start. Setting the busy field of
struct s5p_aes_dev to false made it possible for s5p_tasklet_cb to
change the req field, before s5p_aes_complete was called.
Change the first parameter of s5p_aes_complete to struct
ablkcipher_request. Before spin_unlock, make a copy of the currently
handled request, to ensure s5p_aes_complete function call with the
correct request.
Signed-off-by: Christoph Manszewski <c.manszewski@samsung.com>
Acked-by: Kamil Konieczny <k.konieczny@partner.samsung.com>
Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'crypto')
0 files changed, 0 insertions, 0 deletions