On Sun, Jul 17, 2016 at 10:06 PM, Stephen Rothwell <[email protected]> wrote: > Hi Kees, > > On Sun, 17 Jul 2016 21:49:40 -0700 Kees Cook <[email protected]> wrote: >> >> If I'm reading correctly, this second fixup is wrong. It should read; >> >> kasan_check_read(from, n); >> check_object_size(from, n, true); >> return __arch_copy_to_user(to, from, n); >> >> (i.e. fix double space between "return" and "__arch_copy..." in both >> chunks and add check_object_size() calls after the kasan calls in both >> chunks. > > Yep, sorry. I will fix it up tomorrow.
Cool, thanks! :) -Kees -- Kees Cook Brillo & Chrome OS Security
