diff options
author | Amol Lad <amol@verismonetworks.com> | 2006-12-08 05:40:11 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.osdl.org> | 2006-12-08 11:29:04 -0500 |
commit | 1b3349fa448ce41eda5ae0e6d4dc52837d14ad11 (patch) | |
tree | 3c9e54e9227676a85239b1ca869045cd5108ef2a /drivers/video | |
parent | 1bedb0b2f60dd858fd12e5dd5681fa5da55175f5 (diff) |
[PATCH] ioremap balanced with iounmap for drivers/video/fm2fb
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/fm2fb.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/video/fm2fb.c b/drivers/video/fm2fb.c index 998374cfae6d..70ff55b14596 100644 --- a/drivers/video/fm2fb.c +++ b/drivers/video/fm2fb.c | |||
@@ -283,6 +283,7 @@ static int __devinit fm2fb_probe(struct zorro_dev *z, | |||
283 | 283 | ||
284 | if (register_framebuffer(info) < 0) { | 284 | if (register_framebuffer(info) < 0) { |
285 | fb_dealloc_cmap(&info->cmap); | 285 | fb_dealloc_cmap(&info->cmap); |
286 | iounmap(info->screen_base); | ||
286 | framebuffer_release(info); | 287 | framebuffer_release(info); |
287 | zorro_release_device(z); | 288 | zorro_release_device(z); |
288 | return -EINVAL; | 289 | return -EINVAL; |