diff options
author | Andres Salomon <dilinger@queued.net> | 2009-03-31 18:25:20 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-04-01 11:59:30 -0400 |
commit | d4bc4e8af0a4a34c713f8c1a33a78cedffe8e0b7 (patch) | |
tree | 564a2cc2de3f7b87079a9945c701ebd3b107cf6d /drivers/video | |
parent | 6a7f2829b5f8be124e168265f176dbbbea8861a0 (diff) |
drivers/video/sgivwfb.c: fix memory leaks in removal path
We were leaking both the cmap memory and the info struct memory.
Signed-off-by: Andres Salomon <dilinger@debian.org>
Acked-by: Krzysztof Helt <krzysztof.h1@wp.pl>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/video')
-rw-r--r-- | drivers/video/sgivwfb.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/video/sgivwfb.c b/drivers/video/sgivwfb.c index f5252c2552fd..bba53714a7b1 100644 --- a/drivers/video/sgivwfb.c +++ b/drivers/video/sgivwfb.c | |||
@@ -837,6 +837,8 @@ static int sgivwfb_remove(struct platform_device *dev) | |||
837 | iounmap(par->regs); | 837 | iounmap(par->regs); |
838 | iounmap(info->screen_base); | 838 | iounmap(info->screen_base); |
839 | release_mem_region(DBE_REG_PHYS, DBE_REG_SIZE); | 839 | release_mem_region(DBE_REG_PHYS, DBE_REG_SIZE); |
840 | fb_dealloc_cmap(&info->cmap); | ||
841 | framebuffer_release(info); | ||
840 | } | 842 | } |
841 | return 0; | 843 | return 0; |
842 | } | 844 | } |