Thread (5 messages) flat view 5 messages, 4 authors, 2018-08-03

Re: [PATCH] ppp: mppe: Remove VLA usage

From: Kees Cook <hidden>
Date: 2018-08-02 23:27:08
Also in: lkml

On Mon, Jul 16, 2018 at 4:01 AM, Arnd Bergmann [off-list ref] wrote:
On Mon, Jul 16, 2018 at 6:05 AM, Kees Cook [off-list ref] wrote:
quoted
In the quest to remove all stack VLA usage from the kernel[1], this
removes the discouraged use of AHASH_REQUEST_ON_STACK (and associated
VLA) by switching to shash directly and keeping the associated descriptor
allocated with the regular state on the heap.

[1] https://lkml.kernel.org/r/CA+55aFzCG-zNmZwX4A2FQpadafLfEzK6CC=qPXydAacU1RqZWA@mail.gmail.com

Signed-off-by: Kees Cook <redacted>
I had concerns at first that this approach might make it slower, but
upon reading through implementation of the shash_ahash_ implementation,
I concluded that it can only be better than before, improving both
performance and stack usage.

Acked-by: Arnd Bergmann <arnd@arndb.de>
Paul,

Is this something you can take via your tree?

Thanks,

-Kees


-- 
Kees Cook
Pixel Security
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help