| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <1298792275.19234.928360@goedel.fjf.gnu.de> Date: Sun, 27 Feb 2011 08:37:55 +0100 From: Frank Heckenbach <f.heckenbach@...soft.de> To: linux-kernel@...r.kernel.org Subject: brk() should check randomize_va_space rather then CONFIG_COMPAT_BRK The brk() syscall checks CONFIG_COMPAT_BRK to decide whether the brk area starts just after the end of the code+bss. Since CONFIG_COMPAT_BRK can be overridden by /proc/sys/kernel/randomize_va_space, it should rather check this variable (which is initialized in mm/memory.c from CONFIG_COMPAT_BRK), see the patch below. I've tested it with an old libc5 binary: Without the patch, if the kernel was configured without CONFIG_COMPAT_BRK, the first malloc() fails with any setting of randomize_va_space. With the patch applied, the program runs correctly with randomize_va_space < 2, but not with randomize_va_space == 2, as it should be. --- linux-2.6.37.2/mm/mmap.c.orig 2011-02-25 00:09:00.000000000 +0100 +++ linux-2.6.37.2/mm/mmap.c 2011-02-26 01:13:53.000000000 +0100 @@ -252,11 +252,11 @@ down_write(&mm->mmap_sem); -#ifdef CONFIG_COMPAT_BRK - min_brk = mm->end_code; -#else - min_brk = mm->start_brk; -#endif + if (randomize_va_space < 2) + min_brk = mm->end_code; + else + min_brk = mm->start_brk; + if (brk < min_brk) goto out; -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists