kmap leak fix for x86_32 kdump

22124c99 · Fernando Luis Vázquez Cao <[email protected]> · 2007-10-18 23:39:14 -07:00

copy_oldmem_page should not return leaving a page frame from the
previous kernel mapped.

Signed-off-by: Fernando Luis Vázquez Cao <[email protected]>
Acked-by: Vivek Goyal <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>

Commit: 22124c9999f00340b062fff740db30187bf18454
Parents: 3305a6bc

Changes

1 files changed · +1 −0

File	Status	Add	Del	Content
arch/x86/kernel/crash_dump_32.c	modified	+1	−0	View file

arch/x86/kernel/crash_dump_32.c

View file

diff --git a/arch/x86/kernel/crash_dump_32.c b/arch/x86/kernel/crash_dump_32.c
index 32e75d0..72d0c56 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,
 		if (!kdump_buf_page) {
 			printk(KERN_WARNING "Kdump: Kdump buffer page not"
 				" allocated\n");
+			kunmap_atomic(vaddr, KM_PTE0);
 			return -EFAULT;
 		}
 		copy_page(kdump_buf_page, vaddr);