- fix-process-crash-caused-by-randomisation-and-64k-pages.patch removed from -mm tree

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

 



The patch titled
     fix process crash caused by randomisation and 64k pages
has been removed from the -mm tree.  Its filename was
     fix-process-crash-caused-by-randomisation-and-64k-pages.patch

This patch was dropped because it was merged into mainline or a subsystem tree

------------------------------------------------------
Subject: fix process crash caused by randomisation and 64k pages
From: James Bottomley <James.Bottomley@xxxxxxxxxxxx>

This bug was seen on ppc64, but it could have occurred on any architecture
with a page size of 64k or above.  The problem is that
fs/binfmt_elf.c:randomize_stack_top() randomizes the stack to within 0x7ff
pages.  On 4k page machines, this is 8MB; on 64k page boxes, this is 128MB.
 The problem is that the new binary layout (selected in
arch_pick_mmap_layout) places the mapping segment 128MB or the stack rlimit
away from the top of the process memory, whichever is larger.  If you chose
an rlimit of less than 128MB (most defaults are in the 8Mb range) then you
can end up having your entire stack randomized away.

The fix is to make randomize_stack_top() only steal at most 8MB, which this
patch does.  However, I have to point out that even with this, your stack
rlimit might not be exactly what you get if it's > 128MB, because you're
still losing the random offset of up to 8MB.

The true fix should be to leave an explicit gap for the randomization plus
a buffer when determining mmap_base, but that would involve fixing all the
architectures.

Cc: Arjan van de Ven <arjan@xxxxxxxxxxxxx>
Cc: Ingo Molnar <mingo@xxxxxxx>
Cc: Andi Kleen <ak@xxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---

 fs/binfmt_elf.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff -puN fs/binfmt_elf.c~fix-process-crash-caused-by-randomisation-and-64k-pages fs/binfmt_elf.c
--- a/fs/binfmt_elf.c~fix-process-crash-caused-by-randomisation-and-64k-pages
+++ a/fs/binfmt_elf.c
@@ -507,7 +507,7 @@ out:
 #define INTERPRETER_ELF 2
 
 #ifndef STACK_RND_MASK
-#define STACK_RND_MASK 0x7ff		/* with 4K pages 8MB of VA */
+#define STACK_RND_MASK (0x7ff >> (PAGE_SHIFT - 12))	/* 8MB of VA */
 #endif
 
 static unsigned long randomize_stack_top(unsigned long stack_top)
_

Patches currently in -mm which might be from James.Bottomley@xxxxxxxxxxxx are

origin.patch
git-scsi-misc.patch
git-scsi-rc-fixes.patch
fix--confusion-in-fusion-driver.patch
introduce-config_has_dma.patch

-
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Kernel Newbies FAQ]     [Kernel Archive]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [Bugtraq]     [Photo]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]

  Powered by Linux