|
Message-ID: <20120311233233.GC10787@dztty> Date: Mon, 12 Mar 2012 00:32:33 +0100 From: Djalal Harouni <tixxdz@...ndz.org> To: Linus Torvalds <torvalds@...ux-foundation.org> Cc: Oleg Nesterov <oleg@...hat.com>, linux-kernel@...r.kernel.org, kernel-hardening@...ts.openwall.com, Andrew Morton <akpm@...ux-foundation.org>, Al Viro <viro@...iv.linux.org.uk>, Alexey Dobriyan <adobriyan@...il.com>, "Eric W. Biederman" <ebiederm@...ssion.com>, Vasiliy Kulikov <segoon@...nwall.com>, Kees Cook <keescook@...omium.org>, Solar Designer <solar@...nwall.com>, WANG Cong <xiyou.wangcong@...il.com>, James Morris <james.l.morris@...cle.com>, linux-security-module@...r.kernel.org, linux-fsdevel@...r.kernel.org, Alan Cox <alan@...rguk.ukuu.org.uk>, Greg KH <gregkh@...uxfoundation.org>, Ingo Molnar <mingo@...e.hu>, Stephen Wilson <wilsons@...rt.ca>, "Jason A. Donenfeld" <Jason@...c4.com> Subject: Re: [PATCH 1/9] exec: add a global execve counter On Sun, Mar 11, 2012 at 03:48:02PM -0700, Linus Torvalds wrote: > On Sun, Mar 11, 2012 at 10:25 AM, Oleg Nesterov <oleg@...hat.com> wrote: > > > > It should be per-process, signal_struct makes more sense. Or may be > > mm_struct. > > I do wonder if we shouldn't just consider the "struct mm_struct" > pointer to *be* the unique exec ID. It's what /proc/pid/mem does, and > it works fine, and allows us to just use a normal pointer as the > unique ID. > > Just increment the mm_count for the thing, and hold a reference to it, > and now you're all done. Please Linus have you checked the: [PATCH 9/9] proc: improve and clean up /proc/<pid>/mem protection That keeping the mm struct wont work, since it will eat memory and the OOM-killer will kill some innocent processes, and the abuse can only be catched by the VFS. What's your opinion on it ? And if we do the same for all the /proc/<pid>/* files and even if you clear the VM of the old process, some other information will still be available, even if it is not useful, this is not consistent. Is it ok if the code keeps different objects related to dead processes alive ? -- tixxdz http://opendz.org
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.