diff options
author | Amol Lad <amol@verismonetworks.com> | 2006-12-08 05:40:06 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.osdl.org> | 2006-12-08 11:29:04 -0500 |
commit | 027b53dd92e3d1a23c56d85e006a9bb5e63314f9 (patch) | |
tree | e1b31bb58e3f0c7edccd0558f210504b3536f534 | |
parent | 9cf2014aad58076aa22faacf27742d9610dea05d (diff) |
[PATCH] ioremap balanced with iounmap for drivers/video/retz3fb
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>
-rw-r--r-- | drivers/video/retz3fb.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/video/retz3fb.c b/drivers/video/retz3fb.c index cf41ff177644..bc7ffc84e185 100644 --- a/drivers/video/retz3fb.c +++ b/drivers/video/retz3fb.c | |||
@@ -1423,8 +1423,10 @@ int __init retz3fb_init(void) | |||
1423 | 1423 | ||
1424 | do_install_cmap(0, fb_info); | 1424 | do_install_cmap(0, fb_info); |
1425 | 1425 | ||
1426 | if (register_framebuffer(fb_info) < 0) | 1426 | if (register_framebuffer(fb_info) < 0) { |
1427 | iounmap(zinfo->base); | ||
1427 | return -EINVAL; | 1428 | return -EINVAL; |
1429 | } | ||
1428 | 1430 | ||
1429 | printk(KERN_INFO "fb%d: %s frame buffer device, using %ldK of " | 1431 | printk(KERN_INFO "fb%d: %s frame buffer device, using %ldK of " |
1430 | "video memory\n", fb_info->node, | 1432 | "video memory\n", fb_info->node, |