Re: [patch v3] move RLIMIT_NPROC check from set_user() to do_execve_common()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Vasiliy Kulikov <segoon@xxxxxxxxxxxx>
Subject: Re: [patch v3] move RLIMIT_NPROC check from set_user() to do_execve_common()
From: James Morris <jmorris@xxxxxxxxx>
Date: Fri, 29 Jul 2011 18:17:31 +1000 (EST)
Cc: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>, kernel-hardening@xxxxxxxxxxxxxxxxxx, Stephen Smalley <sds@xxxxxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, Greg Kroah-Hartman <gregkh@xxxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, "David S. Miller" <davem@xxxxxxxxxxxxx>, Jiri Slaby <jslaby@xxxxxxx>, Alexander Viro <viro@xxxxxxxxxxxxxxxxxx>, linux-fsdevel@xxxxxxxxxxxxxxx, KOSAKI Motohiro <kosaki.motohiro@xxxxxxxxxxxxxx>, Eric Paris <eparis@xxxxxxxxxx>, Willy Tarreau <w@xxxxxx>, Sebastian Krahmer <krahmer@xxxxxxx>
In-reply-to: <20110729081156.GB2488@albatros>
User-agent: Alpine 2.00 (LRH 1167 2008-08-23)

On Fri, 29 Jul 2011, Vasiliy Kulikov wrote:

> +	 * RLIMIT_NPROC limits on user->processes have already been checked
> +	 * in set_user().
>  	 */
> +
>  	alter_cred_subscribers(new, 2);

Please get rid of the extra blank line if you re-spin.


Reviewed-by: James Morris <jmorris@xxxxxxxxx>



-- 
James Morris
<jmorris@xxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

References:
- Re: [kernel-hardening] Re: [PATCH] move RLIMIT_NPROC check from set_user() to do_execve_common()
  - From: Solar Designer
- Re: [kernel-hardening] Re: [PATCH] move RLIMIT_NPROC check from set_user() to do_execve_common()
  - From: Linus Torvalds
- Re: [kernel-hardening] [PATCH] move RLIMIT_NPROC check from set_user() to do_execve_common()
  - From: Solar Designer
- [PATCH] move RLIMIT_NPROC check from set_user() to do_execve_common()
  - From: Vasiliy Kulikov
- Re: [kernel-hardening] [PATCH] move RLIMIT_NPROC check from set_user() to do_execve_common()
  - From: Solar Designer
- Re: [kernel-hardening] [PATCH] move RLIMIT_NPROC check from set_user() to do_execve_common()
  - From: NeilBrown
- Re: [kernel-hardening] [PATCH] move RLIMIT_NPROC check from set_user() to do_execve_common()
  - From: Solar Designer
- Re: [kernel-hardening] [PATCH] move RLIMIT_NPROC check from set_user() to do_execve_common()
  - From: NeilBrown
- [patch v2] move RLIMIT_NPROC check from set_user() to do_execve_common()
  - From: Vasiliy Kulikov
- Re: [patch v2] move RLIMIT_NPROC check from set_user() to do_execve_common()
  - From: Vasiliy Kulikov
- [patch v3] move RLIMIT_NPROC check from set_user() to do_execve_common()
  - From: Vasiliy Kulikov

Prev by Date: [patch v3] move RLIMIT_NPROC check from set_user() to do_execve_common()
Next by Date: fs: take the ACL checks to common code
Previous by thread: [patch v3] move RLIMIT_NPROC check from set_user() to do_execve_common()
Next by thread: Re: [PATCH] move RLIMIT_NPROC check from set_user() to do_execve_common()
Index(es):
- Date
- Thread

[Index of Archives] [Linux Ext4 Filesystem] [Union Filesystem] [Filesystem Testing] [Ceph Users] [Ecryptfs] [AutoFS] [Kernel Newbies] [Share Photos] [Security] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux Cachefs] [Reiser Filesystem] [Linux RAID] [Samba] [Device Mapper] [CEPH Development]