diff options
author | Amol Lad <amol@verismonetworks.com> | 2006-10-01 02:29:25 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-10-01 03:39:32 -0400 |
commit | 65da4d81f48e092f71feaf04bf2ccd096b5a5171 (patch) | |
tree | a6822f8aac500b3ab8dae04a2abc615ec4e187ea /drivers/serial/sunsu.c | |
parent | af907dc8cd4157d629e48533b3400786467340d5 (diff) |
[PATCH] ioremap balanced with iounmap for drivers/serial/sunsu.c
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: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: David S. Miller <davem@sunset.davemloft.net>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers/serial/sunsu.c')
-rw-r--r-- | drivers/serial/sunsu.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/serial/sunsu.c b/drivers/serial/sunsu.c index d3a5aeee73a3..9b3b9aaa6b90 100644 --- a/drivers/serial/sunsu.c +++ b/drivers/serial/sunsu.c | |||
@@ -1499,6 +1499,9 @@ static int __devexit su_remove(struct of_device *dev) | |||
1499 | uart_remove_one_port(&sunsu_reg, &up->port); | 1499 | uart_remove_one_port(&sunsu_reg, &up->port); |
1500 | } | 1500 | } |
1501 | 1501 | ||
1502 | if (up->port.membase) | ||
1503 | of_iounmap(up->port.membase, up->reg_size); | ||
1504 | |||
1502 | dev_set_drvdata(&dev->dev, NULL); | 1505 | dev_set_drvdata(&dev->dev, NULL); |
1503 | 1506 | ||
1504 | return 0; | 1507 | return 0; |