lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 18 May 2020 15:30:05 -0700 From: Jakub Kicinski <kuba@...nel.org> To: Vadim Fedorenko <vfedorenko@...ek.ru> Cc: Boris Pismenny <borisp@...lanox.com>, Aviad Yehezkel <aviadye@...lanox.com>, Daniel Borkmann <daniel@...earbox.net>, netdev@...r.kernel.org Subject: Re: [PATCH] net/tls: fix encryption error checking On Sun, 17 May 2020 02:48:39 +0300 Vadim Fedorenko wrote: > tls_push_record can return -EAGAIN because of tcp layer. In that > case open_rec is already in the tx_record list and should not be > freed. > Also the record size can be more than the size requested to write > in tls_sw_do_sendpage(). That leads to overflow of copied variable > and wrong return code. > > Fixes: d10523d0b3d7 ("net/tls: free the record on encryption error") > Signed-off-by: Vadim Fedorenko <vfedorenko@...ek.ru> Doesn't this return -EAGAIN back to user space? Meaning even tho we queued the user space will try to send it again?
Powered by blists - more mailing lists