diff options
| author | Harald Freudenberger <freude@linux.ibm.com> | 2026-08-31 10:38:36 +0200 |
|---|---|---|
| committer | Vasily Gorbik <gor@linux.ibm.com> | 2026-09-08 15:53:41 +0200 |
| commit | ac1481320110b803ab9b79ab4d2ca11a74fc05f2 (patch) | |
| tree | 165ef3a13ef6851ba6d40612f4d54c7273937a3e /scripts/basic | |
| parent | 330148371401de474b656eaf521861f12ec1a1ce (diff) | |
| download | linux-ac1481320110b803ab9b79ab4d2ca11a74fc05f2.tar.gz linux-ac1481320110b803ab9b79ab4d2ca11a74fc05f2.zip | |
s390/crypto: Fix wrong return code to engine in asynch callbacks
When crypto_finalize_hash_request() or
crypto_finalize_skcipher_request() explicitly completes a request, the
do_one_request callback must return 0 to indicate successful
handling. Returning a negative error code causes the crypto engine to
assume the driver failed to take ownership and triggers a second
completion via crypto_request_complete(), resulting in a double
completion. This pattern occurs in paes_s390.c 4 times and once in
phmac_s390.c.
Fixed in phmac_do_one_request() and all four paes do_one_request
callbacks (ecb, cbc, ctr, xts) by returning 0 after explicit
finalization instead of propagating the error code.
Fixes: 6cd87cb5ef6c ("s390/crypto: Rework protected key AES for true asynch support")
Signed-off-by: Harald Freudenberger <freude@linux.ibm.com>
Reviewed-by: Holger Dengler <dengler@linux.ibm.com>
Cc: stable@vger.kernel.org # 6.16+
Signed-off-by: Heiko Carstens <hca@linux.ibm.com>
Signed-off-by: Vasily Gorbik <gor@linux.ibm.com>
Diffstat (limited to 'scripts/basic')
0 files changed, 0 insertions, 0 deletions
