diff options
author | Fernando Luis Vázquez Cao <fernando@oss.ntt.co.jp> | 2007-10-19 02:39:14 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-10-19 14:53:33 -0400 |
commit | 22124c9999f00340b062fff740db30187bf18454 (patch) | |
tree | e3486a4965def48c38355ce7365ee84d6e89618b /arch/x86/kernel/crash_dump_32.c | |
parent | 3305a6bc2c4dbd8f7fde1b9941ccb260bc6355c2 (diff) |
kmap leak fix for x86_32 kdump
copy_oldmem_page should not return leaving a page frame from the
previous kernel mapped.
Signed-off-by: Fernando Luis Vázquez Cao <fernando@oss.ntt.co.jp>
Acked-by: Vivek Goyal <vgoyal@in.ibm.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'arch/x86/kernel/crash_dump_32.c')
-rw-r--r-- | arch/x86/kernel/crash_dump_32.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/arch/x86/kernel/crash_dump_32.c b/arch/x86/kernel/crash_dump_32.c index 32e75d0731a9..72d0c56c1b48 100644 --- a/arch/x86/kernel/crash_dump_32.c +++ b/arch/x86/kernel/crash_dump_32.c | |||
@@ -47,6 +47,7 @@ ssize_t copy_oldmem_page(unsigned long pfn, char *buf, | |||
47 | if (!kdump_buf_page) { | 47 | if (!kdump_buf_page) { |
48 | printk(KERN_WARNING "Kdump: Kdump buffer page not" | 48 | printk(KERN_WARNING "Kdump: Kdump buffer page not" |
49 | " allocated\n"); | 49 | " allocated\n"); |
50 | kunmap_atomic(vaddr, KM_PTE0); | ||
50 | return -EFAULT; | 51 | return -EFAULT; |
51 | } | 52 | } |
52 | copy_page(kdump_buf_page, vaddr); | 53 | copy_page(kdump_buf_page, vaddr); |