From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1767821AbXCJFCM (ORCPT ); Sat, 10 Mar 2007 00:02:12 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1767823AbXCJFCM (ORCPT ); Sat, 10 Mar 2007 00:02:12 -0500 Received: from mga07.intel.com ([143.182.124.22]:22701 "EHLO azsmga101.ch.intel.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1767821AbXCJFCL (ORCPT ); Sat, 10 Mar 2007 00:02:11 -0500 X-ExtLoop1: 1 X-IronPort-AV: i="4.14,269,1170662400"; d="scan'208"; a="193546875:sNHT19448415" Message-ID: <45F23BB5.8090907@linux.intel.com> Date: Fri, 09 Mar 2007 21:01:41 -0800 From: Arjan van de Ven User-Agent: Thunderbird 1.5 (Windows/20051201) MIME-Version: 1.0 To: Andrew Morton CC: Kees Cook , linux-kernel@vger.kernel.org Subject: Re: [PATCH] proc: maps protection References: <20070307012234.GN9621@outflux.net> <20070306175609.267bd7a9.akpm@linux-foundation.org> <20070307021335.GR9621@outflux.net> <20070306185942.585e7cd1.akpm@linux-foundation.org> <20070307031429.GU9621@outflux.net> <1173241331.3236.10.camel@laptopd505.fenrus.org> <20070308205525.GJ6805@outflux.net> <20070309183006.7ff23eff.akpm@linux-foundation.org> In-Reply-To: <20070309183006.7ff23eff.akpm@linux-foundation.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Andrew Morton wrote: >> On Thu, 8 Mar 2007 12:55:25 -0800 Kees Cook wrote: >> On Tue, Mar 06, 2007 at 08:22:11PM -0800, Arjan van de Ven wrote: >>>> [Adding Cc:lkml] >>>> How about using a reduced check, as is done for fd and environ? This >>>> would allow root-running system monitors to still do their job. >>>> Effectively, this changes the test from "is ptracing" to just "can >>>> ptrace". >>>> >>>> If this still isn't considered safe, I'll add the maps_protect file... >>> btw I consider it an information leak that any user can see which >>> files/libraries any other user and root has mmap'd. (and with glibc's >>> stdio mmap feature that goes even beyond direct mmap to fopen()'d). >>> >>> If root or some other user wants to watch >>> hillary-vs-obama-in-the-mud.avi, no other user has ANY business even >>> seeing that. So at minimum it's a privacy issue showing the filenames... >> So, what's the state of this? Is the reduced "allowed to ptrace" check >> good enough for inclusion? What is needed for some form of this patch >> to be included? I'm happy to try new approaches if I can get some >> further input. > > I just don't know what it will break - we're changing things so that user A > cannot monitor user B's memory maps. I feel that it's sure to break > various people's fancy custom system activity monitoring/logging setups, > and the sort of users who will be affected are, alas, the sort of people > who won't run a kernel with this change in it for another couple of years > yet. except if they run RHEL or FC kernels, in which case they already have that change > Do we actually need to disable the whole interface? If all you're > concerned about is the pathname then perhaps the knob could cause that > pathname to be replaced with "". That'll cause things to > break less seriously and still allows somewhat useful info to be gathered. the problem part is that you can see EXACLTY where glibc is loaded in memory, which effectively defeats address space randomization for local users....