Date: Tue, 27 Sep 2011 15:03:09 -0500 (CDT) From: Christoph Lameter <cl@...two.org> To: Vasiliy Kulikov <segoon@...nwall.com> cc: kernel-hardening@...ts.openwall.com, Pekka Enberg <penberg@...nel.org>, Matt Mackall <mpm@...enic.com>, Andrew Morton <akpm@...ux-foundation.org>, linux-mm@...ck.org, Kees Cook <kees@...ntu.com>, Dave Hansen <dave@...ux.vnet.ibm.com>, Valdis.Kletnieks@...edu, Linus Torvalds <torvalds@...ux-foundation.org>, David Rientjes <rientjes@...gle.com>, Alan Cox <alan@...ux.intel.com>, linux-kernel@...r.kernel.org Subject: Re: [PATCH 2/2] mm: restrict access to /proc/meminfo On Tue, 27 Sep 2011, Vasiliy Kulikov wrote: > On Tue, Sep 27, 2011 at 21:56 +0400, Vasiliy Kulikov wrote: > > /proc/meminfo stores information related to memory pages usage, which > > may be used to monitor the number of objects in specific caches (and/or > > the changes of these numbers). This might reveal private information > > similar to /proc/slabinfo infoleaks. To remove the infoleak, just > > restrict meminfo to root. If it is used by unprivileged daemons, > > meminfo permissions can be altered the same way as slabinfo: > > > > groupadd meminfo > > usermod -a -G meminfo $MONITOR_USER > > chmod g+r /proc/meminfo > > chgrp meminfo /proc/meminfo > > Just to make it clear: since this patch breaks "free", I don't propose > it anymore. Viewing free memory is usually necessary to check on reclaim activities (things otherwise operating normally). "free" memory (in the sense of the memory that an application can still allocate) is not really displayed by free. Wish we had a new free that avoids all the misinterpretations. Meminfo is also requires by vmstat. If we want to go down this route then we need some sort of diagnostic group that a user must be part of in order to allow viewing of basic memory statistics.
Powered by blists - more mailing lists
Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.