[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240207011824.2609030-1-kuba@kernel.org>
Date: Tue, 6 Feb 2024 17:18:17 -0800
From: Jakub Kicinski <kuba@...nel.org>
To: davem@...emloft.net
Cc: netdev@...r.kernel.org,
edumazet@...gle.com,
pabeni@...hat.com,
sd@...asysnail.net,
vadim.fedorenko@...ux.dev,
Jakub Kicinski <kuba@...nel.org>
Subject: [PATCH net 0/7] net: tls: fix some issues with async encryption
Hi!
valis was reporting a race on socket close so I sat down to try to fix it.
I used Sabrina's async crypto debug patch to test... and in the process
run into some of the same issues, and created very similar fixes :(
I didn't realize how many of those patches weren't applied. Once I found
Sabrina's code [1] it turned out to be so similar in fact that I added
her S-o-b's and Co-develop'eds in a semi-haphazard way.
With this series in place all expected tests pass with async crypto.
Sabrina had a few more fixes, but I'll leave those to her, things are
not crashing anymore.
[1] https://lore.kernel.org/netdev/cover.1694018970.git.sd@queasysnail.net/
Jakub Kicinski (6):
net: tls: factor out tls_*crypt_async_wait()
tls: fix race between async notify and socket close
tls: fix race between tx work scheduling and socket close
net: tls: handle backlogging of crypto requests
selftests: tls: use exact comparison in recv_partial
net: tls: fix returned read length with async decrypt
Sabrina Dubroca (1):
net: tls: fix use-after-free with partial reads and async decrypt
include/net/tls.h | 5 --
net/tls/tls_sw.c | 135 ++++++++++++++----------------
tools/testing/selftests/net/tls.c | 8 +-
3 files changed, 66 insertions(+), 82 deletions(-)
--
2.43.0
Powered by blists - more mailing lists