Re: [kernel-hardening] Re: [PATCH v9 1/4] syscalls: Verify address limit before returning to user-mode

From: Borislav Petkov
Date: Thu May 11 2017 - 07:22:43 EST


On Tue, May 09, 2017 at 04:31:00PM -0700, Kees Cook wrote:
> > I don't like silent fixups. If we want to do this, we should BUG or
> > at least WARN, not just change the addr limit. But I'm also not
> > convinced it's indicative of an actual bug here.
>
> Nothing should enter that function with KERNEL_DS set, right?
>
> BUG_ON(get_fs() != USER_DS);

We're feeling triggerhappy, aren't we? A nice juicy WARN-splat along
with a fixup looks much better than killing the box, to me.

--
Regards/Gruss,
Boris.

Good mailing practices for 400: avoid top-posting and trim the reply.