diff options
author | Amol Lad <amol@verismonetworks.com> | 2006-12-08 05:40:12 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.osdl.org> | 2006-12-08 11:29:04 -0500 |
commit | d02abed8aa6bd37abcd802de64a63ed2af5e18ff (patch) | |
tree | 06359af2e06004d258c96aa4c374083ca8673bee /drivers/video | |
parent | 8d4c767ef56473b68a601274612d604c8c334dc9 (diff) |
[PATCH] ioremap balanced with iounmap for drivers/video/cyberfb
ioremap must be balanced by an iounmap and failing to do so can result in a
memory leak.
Signed-off-by: Amol Lad <amol@verismonetworks.com>
Cc: "Antonino A. Daplas" <adaplas@pol.net>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers/video')
-rw-r--r-- | drivers/video/cyberfb.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/video/cyberfb.c b/drivers/video/cyberfb.c index c40e72dafb0e..bc1cc44b1e3f 100644 --- a/drivers/video/cyberfb.c +++ b/drivers/video/cyberfb.c | |||
@@ -1055,6 +1055,8 @@ int __init cyberfb_init(void) | |||
1055 | 1055 | ||
1056 | if (register_framebuffer(&fb_info) < 0) { | 1056 | if (register_framebuffer(&fb_info) < 0) { |
1057 | DPRINTK("EXIT - register_framebuffer failed\n"); | 1057 | DPRINTK("EXIT - register_framebuffer failed\n"); |
1058 | if (CyberBase) | ||
1059 | iounmap(CyberBase); | ||
1058 | release_mem_region(CyberMem_phys, 0x400000); | 1060 | release_mem_region(CyberMem_phys, 0x400000); |
1059 | release_mem_region(CyberRegs_phys, 0x10000); | 1061 | release_mem_region(CyberRegs_phys, 0x10000); |
1060 | return -EINVAL; | 1062 | return -EINVAL; |