oss-sec mailing list archives
CVE request: information leak / potential crash in sys_move_pages
From: Marcus Meissner <meissner () suse de>
Date: Sun, 7 Feb 2010 02:50:21 +0100
Hi, I spotted a problem in sys_move_pages, where "node" value is read from userspace, but not limited to the node set within the kernel itself. Due to the bit tests in mm/migrate.c:do_move_pages it is easy to read out the kernel memory (as node can also be negative). (The node_isset and node_state functions just map to test_bit, which has no limiter in the normal implementations.) There also is (in my eyes) the chance we can corrupt kernel memory later on if we have all the right bits setup, but I did not research this further. Issue was present starting as sys_move_pages was introduced in 2.6.18. Solved in mainline by commit below. Needs a CVE for information leakage at least. Have a nice weekend, Marcus commit 6f5a55f1a6c5abee15a0e878e5c74d9f1569b8b0 Author: Linus Torvalds <torvalds () linux-foundation org> Date: Fri Feb 5 16:16:50 2010 -0800 Fix potential crash with sys_move_pages We incorrectly depended on the 'node_state/node_isset()' functions testing the node range, rather than checking it explicitly. That's not reliable, even if it might often happen to work. So do the proper explicit test. Reported-by: Marcus Meissner <meissner () suse de> Acked-and-tested-by: Brice Goglin <Brice.Goglin () inria fr> Acked-by: Hugh Dickins <hugh.dickins () tiscali co uk> Cc: stable () kernel org Signed-off-by: Linus Torvalds <torvalds () linux-foundation org> diff --git a/mm/migrate.c b/mm/migrate.c index efddbf0..9a0db5b 100644 --- a/mm/migrate.c +++ b/mm/migrate.c @@ -912,6 +912,9 @@ static int do_pages_move(struct mm_struct *mm, struct task_struct *task, goto out_pm; err = -ENODEV; + if (node < 0 || node >= MAX_NUMNODES) + goto out_pm; + if (!node_state(node, N_HIGH_MEMORY)) goto out_pm;
Current thread:
- CVE request: information leak / potential crash in sys_move_pages Marcus Meissner (Feb 06)
- Re: CVE request: information leak / potential crash in sys_move_pages Eugene Teo (Feb 07)
- Re: CVE request: information leak / potential crash in sys_move_pages Marcus Meissner (Feb 08)