[9267] in bugtraq

home help back first fref pref prev next nref lref last post

Re: 2.2.0 SECURITY (fwd)

daemon@ATHENA.MIT.EDU (Alan Cox)
Wed Jan 27 18:33:14 1999

Date: 	Wed, 27 Jan 1999 19:19:01 +0000
Reply-To: Alan Cox <alan@LXORGUK.UKUU.ORG.UK>
From: Alan Cox <alan@LXORGUK.UKUU.ORG.UK>
X-To:         stany@PET.NOTBSD.ORG
To: BUGTRAQ@NETSPACE.ORG
In-Reply-To:  <Pine.LNX.3.96.990127031140.2923A-100000@pet.notbsd.org> from
              "//Stany" at Jan 27, 99 03:14:29 am

> Later on down the thread there are other people then Dan Burcaw who say
> that the bug crashes their boxes too.
>
> I suggest reading the thread on Linux Kernel archives for more
> information.  See information at the bottof of forwarded message for
> information.

Yep its real bug. The relevant info for early adopters is:


From:	MOLNAR Ingo <mingo@chiara.csoma.elte.hu>
To:	linux-kernel@vger.rutgers.edu, linux-smp@vger.rutgers.edu
cc:	Linus Torvalds <torvalds@transmeta.com>, Alan Cox <alan@lxorguk.ukuu.org.uk>, "David S. Miller" <davem@dm.cobaltmicro.com>
Subject: [patch] 'coredump crash' fixed

it was a very subtle bug and has nothing to do with coredumps at all, but
it's very rare and the invalid coredump ELF layout accidentally triggered
the bug.

with the attached patch applied i get:

[root@moon /root]# ldd core
        not a dynamic executable
[root@moon /root]#

just as expected. The reason why we crash and why it made the kernel
reboot in such a nasty way was that munmap() did just a tad more work than
necessary and we zapped 0xc0000000's page table entry ... that is a pretty
vital piece of 4M virtual space on Linux ;) (erm, just dont ask me how i
managed to debug this ;)

-- mingo, running a hopefully much safer kernel now ;)

--- linux/mm/mmap.c.orig	Wed Jan 27 14:09:31 1999
+++ linux/mm/mmap.c	Wed Jan 27 14:06:09 1999
@@ -558,7 +558,7 @@
 	unsigned long start, unsigned long end)
 {
 	unsigned long first = start & PGDIR_MASK;
-	unsigned long last = (end & PGDIR_MASK) + PGDIR_SIZE;
+	unsigned long last = ((end-1) & PGDIR_MASK) + PGDIR_SIZE;

 	if (!prev) {
 		prev = mm->mmap;

-
Linux SMP list: FIRST see FAQ at http://www.irisa.fr/prive/mentre/smp-faq/
To Unsubscribe: send "unsubscribe linux-smp" to majordomo@vger.rutgers.edu

home help back first fref pref prev next nref lref last post