netdev - Re: [PATCH net-next] tls: cap the output scatter list to something reasonable

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 linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20220217174850.1498c61c@kicinski-fedora-pc1c0hjn.dhcp.thefacebook.com>
Date:   Thu, 17 Feb 2022 17:48:50 -0800
From:   Jakub Kicinski <kuba@...nel.org>
To:     Al Viro <viro@...iv.linux.org.uk>
Cc:     davem@...emloft.net, netdev@...r.kernel.org, borisp@...dia.com,
        john.fastabend@...il.com, daniel@...earbox.net,
        vfedorenko@...ek.ru, kernel-team@...com, axboe@...nel.dk
Subject: Re: [PATCH net-next] tls: cap the output scatter list to something
 reasonable

On Mon, 7 Feb 2022 15:34:50 -0800 Jakub Kicinski wrote:
> > What's the point of pass maxpages as argument to that, seeing that
> > you ignore the value you've got?  I'm just trying to understand what
> > semantics do you really intend for that thing.
> > 
> > Another thing: looking at that bunch now, for pipe-backed ones
> > that won't work.  It's a bug, strictly speaking, even though
> > the actual primitives that grab those pages *will* honour the
> > truncation/reexpand.
> > 
> > Frankly, I wonder if we would be better off with making
> > iov_iter_npages() a wrapper for that one, passing SIZE_MAX as
> > maxbytes.  How does the following (completely untested) look for you?  
> 
> That looks cleaner to me as well! Will you submit officially or should 
> I take care of the conversion?  My v1 has already made its way to
> net-next.

Gentle reminder that I need at least your sign-off tag to make progress
here.