diff options
author | Amol Lad <amol@verismonetworks.com> | 2006-12-08 05:40:04 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.osdl.org> | 2006-12-08 11:29:04 -0500 |
commit | e4bf051b4f3121c5abdde836b1429e4a4459adb4 (patch) | |
tree | 3325ae91ab700fb170f43f188789c6b6131acb8d /drivers/video/tgafb.c | |
parent | a02f6402d5a18f5a4fd35cb82887c1097d0f7792 (diff) |
[PATCH] ioremap balanced with iounmap for drivers/video/tgafb
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/tgafb.c')
-rw-r--r-- | drivers/video/tgafb.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/video/tgafb.c b/drivers/video/tgafb.c index 08b3153bfb3f..4b88fab83b74 100644 --- a/drivers/video/tgafb.c +++ b/drivers/video/tgafb.c | |||
@@ -1439,6 +1439,8 @@ tgafb_pci_register(struct pci_dev *pdev, const struct pci_device_id *ent) | |||
1439 | return 0; | 1439 | return 0; |
1440 | 1440 | ||
1441 | err1: | 1441 | err1: |
1442 | if (mem_base) | ||
1443 | iounmap(mem_base); | ||
1442 | release_mem_region(bar0_start, bar0_len); | 1444 | release_mem_region(bar0_start, bar0_len); |
1443 | err0: | 1445 | err0: |
1444 | kfree(all); | 1446 | kfree(all); |