]> git.baikalelectronics.ru Git - kernel.git/commit
x86_64, entry: Fix out of bounds read on sysenter
authorAndy Lutomirski <luto@amacapital.net>
Sat, 1 Nov 2014 01:08:45 +0000 (18:08 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Sat, 1 Nov 2014 01:47:09 +0000 (18:47 -0700)
commitd44b612dc37c2e6760798869f5e9670d4069bc5f
treec1e404d37868318063fcb6f57d40346b0c1a131d
parentab5abf4854aa8c998bd16ff5fa3ed4745ace257b
x86_64, entry: Fix out of bounds read on sysenter

Rusty noticed a Really Bad Bug (tm) in my NT fix.  The entry code
reads out of bounds, causing the NT fix to be unreliable.  But, and
this is much, much worse, if your stack is somehow just below the
top of the direct map (or a hole), you read out of bounds and crash.

Excerpt from the crash:

[    1.129513] RSP: 0018:ffff88001da4bf88  EFLAGS: 00010296

  2b:*    f7 84 24 90 00 00 00     testl  $0x4000,0x90(%rsp)

That read is deterministically above the top of the stack.  I
thought I even single-stepped through this code when I wrote it to
check the offset, but I clearly screwed it up.

Fixes: ab5ffae05793 ("x86_64, entry: Filter RFLAGS.NT on entry from userspace")
Reported-by: Rusty Russell <rusty@ozlabs.org>
Cc: stable@vger.kernel.org
Signed-off-by: Andy Lutomirski <luto@amacapital.net>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
arch/x86/ia32/ia32entry.S