Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934798AbdC3Tlw (ORCPT ); Thu, 30 Mar 2017 15:41:52 -0400 Received: from scorn.kernelslacker.org ([45.56.101.199]:54684 "EHLO scorn.kernelslacker.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934091AbdC3Tlt (ORCPT ); Thu, 30 Mar 2017 15:41:49 -0400 Date: Thu, 30 Mar 2017 15:41:43 -0400 From: Dave Jones To: Kees Cook Cc: Tommi Rantala , Linux-MM , LKML , Laura Abbott , Ingo Molnar , Josh Poimboeuf , Mark Rutland , Eric Biggers Subject: Re: sudo x86info -a => kernel BUG at mm/usercopy.c:78! Message-ID: <20170330194143.cbracica3w3ijrcx@codemonkey.org.uk> Mail-Followup-To: Dave Jones , Kees Cook , Tommi Rantala , Linux-MM , LKML , Laura Abbott , Ingo Molnar , Josh Poimboeuf , Mark Rutland , Eric Biggers References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20170306 (1.8.0) X-Spam-Note: SpamAssassin invocation failed Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1156 Lines: 34 On Thu, Mar 30, 2017 at 09:45:26AM -0700, Kees Cook wrote: > On Wed, Mar 29, 2017 at 11:44 PM, Tommi Rantala > wrote: > > Hi, > > > > Running: > > > > $ sudo x86info -a > > > > On this HP ZBook 15 G3 laptop kills the x86info process with segfault and > > produces the following kernel BUG. > > > > $ git describe > > v4.11-rc4-40-gfe82203 > > > > It is also reproducible with the fedora kernel: 4.9.14-200.fc25.x86_64 > > > > Full dmesg output here: https://pastebin.com/raw/Kur2mpZq > > > > [ 51.418954] usercopy: kernel memory exposure attempt detected from > > ffff880000090000 (dma-kmalloc-256) (4096 bytes) > > This seems like a real exposure: the copy is attempting to read 4096 > bytes from a 256 byte object. The code[1] is doing a 4k read from /dev/mem in the range 0x90000 -> 0xa0000 According to arch/x86/mm/init.c:devmem_is_allowed, that's still valid.. Note that the printk is using the direct mapping address. Is that what's being passed down to devmem_is_allowed now ? If so, that's probably what broke. Dave [1] https://github.com/kernelslacker/x86info/blob/master/mptable.c