public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* Make LSM_MMAP_MIN_ADDR default match its help text.
       [not found] <200908172100.n7HL08SR001061@hera.kernel.org>
@ 2009-08-18 17:47 ` Dave Jones
  2009-08-18 17:50   ` Eric Paris
  2009-08-18 20:14   ` Andreas Schwab
  0 siblings, 2 replies; 5+ messages in thread
From: Dave Jones @ 2009-08-18 17:47 UTC (permalink / raw)
  To: Linux Kernel Mailing List; +Cc: Eric Paris, James Morris

Commit 788084aba2ab7348257597496befcbccabdc98a3 added the LSM_MMAP_MIN_ADDR
option, whose help text states "For most ia64, ppc64 and x86 users with lots
of address space a value of 65536 is reasonable and should cause no problems."
Which implies that it's default setting was typoed.

Signed-off-by: Dave Jones <davej@redhat.com>

--- linux-2.6.30.noarch/security/Kconfig~	2009-08-18 13:45:33.000000000 -0400
+++ linux-2.6.30.noarch/security/Kconfig	2009-08-18 13:45:36.000000000 -0400
@@ -116,7 +116,7 @@ config SECURITY_ROOTPLUG
 config LSM_MMAP_MIN_ADDR
 	int "Low address space for LSM to from user allocation"
 	depends on SECURITY && SECURITY_SELINUX
-	default 65535
+	default 65536
 	help
 	  This is the portion of low virtual memory which should be protected
 	  from userspace allocation.  Keeping a user from writing to low pages

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Make LSM_MMAP_MIN_ADDR default match its help text.
  2009-08-18 17:47 ` Make LSM_MMAP_MIN_ADDR default match its help text Dave Jones
@ 2009-08-18 17:50   ` Eric Paris
  2009-08-18 20:14   ` Andreas Schwab
  1 sibling, 0 replies; 5+ messages in thread
From: Eric Paris @ 2009-08-18 17:50 UTC (permalink / raw)
  To: Dave Jones; +Cc: Linux Kernel Mailing List, James Morris

On Tue, 2009-08-18 at 13:47 -0400, Dave Jones wrote:
> Commit 788084aba2ab7348257597496befcbccabdc98a3 added the LSM_MMAP_MIN_ADDR
> option, whose help text states "For most ia64, ppc64 and x86 users with lots
> of address space a value of 65536 is reasonable and should cause no problems."
> Which implies that it's default setting was typoed.

Acked-by: Eric Paris <eparis@redhat.com>

This can wait or go now.  the tests against this are always a strict >
(not >=) and they are always rounded to PAGE_SIZE.  Thus both have the
same address protections.

-Eric

> Signed-off-by: Dave Jones <davej@redhat.com>
> 
> --- linux-2.6.30.noarch/security/Kconfig~	2009-08-18 13:45:33.000000000 -0400
> +++ linux-2.6.30.noarch/security/Kconfig	2009-08-18 13:45:36.000000000 -0400
> @@ -116,7 +116,7 @@ config SECURITY_ROOTPLUG
>  config LSM_MMAP_MIN_ADDR
>  	int "Low address space for LSM to from user allocation"
>  	depends on SECURITY && SECURITY_SELINUX
> -	default 65535
> +	default 65536
>  	help
>  	  This is the portion of low virtual memory which should be protected
>  	  from userspace allocation.  Keeping a user from writing to low pages



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Make LSM_MMAP_MIN_ADDR default match its help text.
  2009-08-18 17:47 ` Make LSM_MMAP_MIN_ADDR default match its help text Dave Jones
  2009-08-18 17:50   ` Eric Paris
@ 2009-08-18 20:14   ` Andreas Schwab
  2009-08-18 20:22     ` Eric Paris
  2009-08-18 20:27     ` Dave Jones
  1 sibling, 2 replies; 5+ messages in thread
From: Andreas Schwab @ 2009-08-18 20:14 UTC (permalink / raw)
  To: Dave Jones; +Cc: Linux Kernel Mailing List, Eric Paris, James Morris

Dave Jones <davej@redhat.com> writes:

>  config LSM_MMAP_MIN_ADDR
>  	int "Low address space for LSM to from user allocation"

This sentence no verb.

Andreas.
---
>From 4ba5116764386ceb1a9da4cb6800defd84f0e3c1 Mon Sep 17 00:00:00 2001
From: Andreas Schwab <schwab@linux-m68k.org>
Date: Tue, 18 Aug 2009 22:08:43 +0200
Subject: [PATCH] Fix prompt for LSM_MMAP_MIN_ADDR.

Signed-off-by: Andreas Schwab <schwab@linux-m68k.org>
---
 security/Kconfig |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/security/Kconfig b/security/Kconfig
index 9c60c34..4d1a2e3 100644
--- a/security/Kconfig
+++ b/security/Kconfig
@@ -114,7 +114,7 @@ config SECURITY_ROOTPLUG
 	  If you are unsure how to answer this question, answer N.
 
 config LSM_MMAP_MIN_ADDR
-	int "Low address space for LSM to from user allocation"
+	int "Low address space for LSM to protect from user allocation"
 	depends on SECURITY && SECURITY_SELINUX
 	default 65535
 	help
-- 
1.6.4

-- 
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: Make LSM_MMAP_MIN_ADDR default match its help text.
  2009-08-18 20:14   ` Andreas Schwab
@ 2009-08-18 20:22     ` Eric Paris
  2009-08-18 20:27     ` Dave Jones
  1 sibling, 0 replies; 5+ messages in thread
From: Eric Paris @ 2009-08-18 20:22 UTC (permalink / raw)
  To: Andreas Schwab; +Cc: Dave Jones, Linux Kernel Mailing List, James Morris

On Tue, 2009-08-18 at 22:14 +0200, Andreas Schwab wrote:
> Dave Jones <davej@redhat.com> writes:
> 
> >  config LSM_MMAP_MIN_ADDR
> >  	int "Low address space for LSM to from user allocation"
> 
> This sentence no verb.
> 
> Andreas.
> ---
> From 4ba5116764386ceb1a9da4cb6800defd84f0e3c1 Mon Sep 17 00:00:00 2001
> From: Andreas Schwab <schwab@linux-m68k.org>
> Date: Tue, 18 Aug 2009 22:08:43 +0200
> Subject: [PATCH] Fix prompt for LSM_MMAP_MIN_ADDR.
> 
> Signed-off-by: Andreas Schwab <schwab@linux-m68k.org>

Acked-by: Eric Paris <eparis@redhat.com>



> ---
>  security/Kconfig |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/security/Kconfig b/security/Kconfig
> index 9c60c34..4d1a2e3 100644
> --- a/security/Kconfig
> +++ b/security/Kconfig
> @@ -114,7 +114,7 @@ config SECURITY_ROOTPLUG
>  	  If you are unsure how to answer this question, answer N.
>  
>  config LSM_MMAP_MIN_ADDR
> -	int "Low address space for LSM to from user allocation"
> +	int "Low address space for LSM to protect from user allocation"
>  	depends on SECURITY && SECURITY_SELINUX
>  	default 65535
>  	help
> -- 
> 1.6.4
> 



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Make LSM_MMAP_MIN_ADDR default match its help text.
  2009-08-18 20:14   ` Andreas Schwab
  2009-08-18 20:22     ` Eric Paris
@ 2009-08-18 20:27     ` Dave Jones
  1 sibling, 0 replies; 5+ messages in thread
From: Dave Jones @ 2009-08-18 20:27 UTC (permalink / raw)
  To: Andreas Schwab; +Cc: Linux Kernel Mailing List, Eric Paris, James Morris

On Tue, Aug 18, 2009 at 10:14:29PM +0200, Andreas Schwab wrote:
 > Dave Jones <davej@redhat.com> writes:
 > 
 > >  config LSM_MMAP_MIN_ADDR
 > >  	int "Low address space for LSM to from user allocation"
 > 
 > This sentence no verb.

Oops. I accidentally that.

Thanks.

	Dave 


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2009-08-18 20:28 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <200908172100.n7HL08SR001061@hera.kernel.org>
2009-08-18 17:47 ` Make LSM_MMAP_MIN_ADDR default match its help text Dave Jones
2009-08-18 17:50   ` Eric Paris
2009-08-18 20:14   ` Andreas Schwab
2009-08-18 20:22     ` Eric Paris
2009-08-18 20:27     ` Dave Jones

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox