Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753119AbdGFSuR (ORCPT ); Thu, 6 Jul 2017 14:50:17 -0400 Received: from mail-it0-f47.google.com ([209.85.214.47]:36838 "EHLO mail-it0-f47.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752675AbdGFSuO (ORCPT ); Thu, 6 Jul 2017 14:50:14 -0400 MIME-Version: 1.0 In-Reply-To: <1499363602.26846.3.camel@redhat.com> References: <20170706002718.GA102852@beast> <1499363602.26846.3.camel@redhat.com> From: Kees Cook Date: Thu, 6 Jul 2017 11:50:07 -0700 X-Google-Sender-Auth: tA_rst0dFjwlmbdXKGHQb-6Ax2o Message-ID: Subject: Re: [PATCH v3] mm: Add SLUB free list pointer obfuscation To: Rik van Riel Cc: Christoph Lameter , Andrew Morton , Pekka Enberg , David Rientjes , Joonsoo Kim , "Paul E. McKenney" , Ingo Molnar , Josh Triplett , Andy Lutomirski , Nicolas Pitre , Tejun Heo , Daniel Mack , Sebastian Andrzej Siewior , Sergey Senozhatsky , Helge Deller , Linux-MM , Tycho Andersen , LKML , "kernel-hardening@lists.openwall.com" Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2730 Lines: 63 On Thu, Jul 6, 2017 at 10:53 AM, Rik van Riel wrote: > On Thu, 2017-07-06 at 10:55 -0500, Christoph Lameter wrote: >> On Thu, 6 Jul 2017, Kees Cook wrote: >> > That requires a series of arbitrary reads. This is protecting >> > against >> > attacks that use an adjacent slab object write overflow to write >> > the >> > freelist pointer. This internal structure is very reliable, and has >> > been the basis of freelist attacks against the kernel for a decade. >> >> These reads are not arbitrary. You can usually calculate the page struct >> address easily from the address and then do a couple of loads to get >> there. >> >> Ok so you get rid of the old attacks because we did not have that >> hardening in effect when they designed their approaches? > > The hardening protects against situations where > people do not have arbitrary code execution and > memory read access in the kernel, with the goal > of protecting people from acquiring those abilities. Right. This is about blocking the escalation of attack capability. For slab object overflow flaws, there are mainly two exploitation methods: adjacent allocated object overwrite and adjacent freed object overwrite (i.e. a freelist pointer overwrite). The first attack depends heavily on which slab cache (and therefore which structures) has been exposed by the bug. It's a very narrow and specific attack method. The freelist attack is entirely general purpose since it provides a reliable way to gain arbitrary write capabilities. Protecting against that attack greatly narrows the options for an attacker which makes attacks more expensive to create and possibly less reliable (and reliability is crucial to successful attacks). >> > It is a probabilistic defense, but then so is the stack protector. >> > This is a similar defense; while not perfect it makes the class of >> > attack much more difficult to mount. >> >> Na I am not convinced of the "much more difficult". Maybe they will just >> have to upgrade their approaches to fetch the proper values to >> decode. > > Easier said than done. Most of the time there is an > unpatched vulnerability outstanding, there is only > one known issue, before the kernel is updated by the > user, to a version that does not have that issue. > > Bypassing kernel hardening typically requires the > use of multiple vulnerabilities, and the absence of > roadblocks (like hardening) that make a type of > vulnerability exploitable. > > Between usercopy hardening, and these slub freelist > canaries (which is what they effectively are), several > classes of exploits are no longer usable. Yup. I've been thinking of this patch kind of like glibc's PTR_MANGLE feature. -Kees -- Kees Cook Pixel Security