Re: VLA removal (was Re: [RFC 2/2] lustre: use VLA_SAFE)

2018-03-07 Thread Daniel Micay
On 7 March 2018 at 13:09, Linus Torvalds wrote: > On Wed, Mar 7, 2018 at 9:37 AM, Kees Cook wrote: >> >> Building with -Wvla, I see 209 unique locations reported in 60 directories: >> http://paste.ubuntu.com/p/srQxwPQS9s/ > > Ok, that's not so bad. Maybe Greg could even add it to one of those > t

Re: VLA removal (was Re: [RFC 2/2] lustre: use VLA_SAFE)

2018-03-07 Thread Linus Torvalds
On Wed, Mar 7, 2018 at 9:37 AM, Kees Cook wrote: > > Building with -Wvla, I see 209 unique locations reported in 60 directories: > http://paste.ubuntu.com/p/srQxwPQS9s/ Ok, that's not so bad. Maybe Greg could even add it to one of those things he encourages new people to do? Because at least *so

VLA removal (was Re: [RFC 2/2] lustre: use VLA_SAFE)

2018-03-07 Thread Kees Cook
On Wed, Mar 7, 2018 at 2:10 AM, Tobin C. Harding wrote: > On Tue, Mar 06, 2018 at 09:46:02PM -0800, Kees Cook wrote: >> On Tue, Mar 6, 2018 at 9:27 PM, Tobin C. Harding wrote: >> > Currently lustre uses a VLA to store a string on the stack. We can use >> > the newly define VLA_SAFE macro to make