Re: [PATCH] usercopy: add testcases to check zeroing on failure of usercopy

2017-02-15 Thread Kees Cook
On Wed, Feb 15, 2017 at 2:45 AM, Mark Rutland wrote: > On Tue, Feb 14, 2017 at 12:36:48PM -0800, Kees Cook wrote: >> On Mon, Feb 13, 2017 at 5:44 PM, Hoeun Ryu wrote: >> >> On Feb 14, 2017, at 4:24 AM, Kees Cook wrote: >> >>> On Mon, Feb 13, 2017 at 10:33 AM, Kees Cook >> >>> wrote: >> On

Re: [PATCH] usercopy: add testcases to check zeroing on failure of usercopy

2017-02-15 Thread Mark Rutland
On Tue, Feb 14, 2017 at 12:36:48PM -0800, Kees Cook wrote: > On Mon, Feb 13, 2017 at 5:44 PM, Hoeun Ryu wrote: > >> On Feb 14, 2017, at 4:24 AM, Kees Cook wrote: > >>> On Mon, Feb 13, 2017 at 10:33 AM, Kees Cook wrote: > On Sat, Feb 11, 2017 at 10:13 PM, Hoeun Ryu wrote: > @@ -69,25

Re: [PATCH] usercopy: add testcases to check zeroing on failure of usercopy

2017-02-14 Thread Hoeun Ryu
> On Feb 15, 2017, at 5:36 AM, Kees Cook wrote: > >> On Mon, Feb 13, 2017 at 5:44 PM, Hoeun Ryu wrote: >> >> On Feb 14, 2017, at 4:24 AM, Kees Cook wrote: > On Mon, Feb 13, 2017 at 10:33 AM, Kees Cook wrote: > On Sat, Feb 11, 2017 at 10:13 PM, Hoeun Ryu wrote: > In t

Re: [PATCH] usercopy: add testcases to check zeroing on failure of usercopy

2017-02-14 Thread Kees Cook
On Mon, Feb 13, 2017 at 5:44 PM, Hoeun Ryu wrote: > > >> On Feb 14, 2017, at 4:24 AM, Kees Cook wrote: >> >>> On Mon, Feb 13, 2017 at 10:33 AM, Kees Cook wrote: On Sat, Feb 11, 2017 at 10:13 PM, Hoeun Ryu wrote: In the hardend usercopy, the destination buffer will be zeroed if co

Re: [PATCH] usercopy: add testcases to check zeroing on failure of usercopy

2017-02-13 Thread Hoeun Ryu
> On Feb 14, 2017, at 4:24 AM, Kees Cook wrote: > >> On Mon, Feb 13, 2017 at 10:33 AM, Kees Cook wrote: >>> On Sat, Feb 11, 2017 at 10:13 PM, Hoeun Ryu wrote: >>> In the hardend usercopy, the destination buffer will be zeroed if >>> copy_from_user/get_user fails. This patch adds testcases for

Re: [PATCH] usercopy: add testcases to check zeroing on failure of usercopy

2017-02-13 Thread Kees Cook
On Mon, Feb 13, 2017 at 10:33 AM, Kees Cook wrote: > On Sat, Feb 11, 2017 at 10:13 PM, Hoeun Ryu wrote: >> In the hardend usercopy, the destination buffer will be zeroed if >> copy_from_user/get_user fails. This patch adds testcases for it. >> The destination buffer is set with non-zero value bef

Re: [PATCH] usercopy: add testcases to check zeroing on failure of usercopy

2017-02-13 Thread Kees Cook
On Sat, Feb 11, 2017 at 10:13 PM, Hoeun Ryu wrote: > In the hardend usercopy, the destination buffer will be zeroed if > copy_from_user/get_user fails. This patch adds testcases for it. > The destination buffer is set with non-zero value before illegal > copy_from_user/get_user is executed and the

Re: [PATCH] usercopy: add testcases to check zeroing on failure of usercopy

2017-02-13 Thread Kees Cook
On Sat, Feb 11, 2017 at 10:13 PM, Hoeun Ryu wrote: > In the hardend usercopy, the destination buffer will be zeroed if > copy_from_user/get_user fails. This patch adds testcases for it. > The destination buffer is set with non-zero value before illegal > copy_from_user/get_user is executed and the