diff options
| author | Andres Salomon <dilinger@queued.net> | 2009-04-01 00:25:20 +0200 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-04-01 17:59:30 +0200 |
| commit | d4bc4e8af0a4a34c713f8c1a33a78cedffe8e0b7 (patch) | |
| tree | 564a2cc2de3f7b87079a9945c701ebd3b107cf6d | |
| parent | fbdev: uninline lock_fb_info() (diff) | |
| download | linux-d4bc4e8af0a4a34c713f8c1a33a78cedffe8e0b7.tar.xz linux-d4bc4e8af0a4a34c713f8c1a33a78cedffe8e0b7.zip | |
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>
| -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) iounmap(par->regs); iounmap(info->screen_base); release_mem_region(DBE_REG_PHYS, DBE_REG_SIZE); + fb_dealloc_cmap(&info->cmap); + framebuffer_release(info); } return 0; } |