diff options
| author | Julia Lawall <Julia.Lawall@lip6.fr> | 2012-01-12 00:55:10 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2012-01-13 05:05:28 +0100 |
| commit | 25cecd7e355bf90b8fea039bd06947bb3234e77d (patch) | |
| tree | 287848d194e82ba41fb71d5e4c1ac340a80b6d05 /drivers | |
| parent | drivers/net/ethernet/tundra/tsi108_eth.c: add missing iounmap (diff) | |
| download | linux-25cecd7e355bf90b8fea039bd06947bb3234e77d.tar.xz linux-25cecd7e355bf90b8fea039bd06947bb3234e77d.zip | |
drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c: add missing iounmap
Add missing iounmap in error handling code, in a case where the function
already preforms iounmap on some other execution path.
A simplified version of the semantic match that finds this problem is as
follows: (http://coccinelle.lip6.fr/)
// <smpl>
@@
expression e;
statement S,S1;
int ret;
@@
e = \(ioremap\|ioremap_nocache\)(...)
... when != iounmap(e)
if (<+...e...+>) S
... when any
when != iounmap(e)
*if (...)
{ ... when != iounmap(e)
return ...; }
... when any
iounmap(e);
// </smpl>
Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c index 7b1594f4944e..1ac83243649a 100644 --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c @@ -62,7 +62,7 @@ static int stmmac_pltfr_probe(struct platform_device *pdev) priv = stmmac_dvr_probe(&(pdev->dev), plat_dat); if (!priv) { pr_err("%s: main drivr probe failed", __func__); - goto out_release_region; + goto out_unmap; } priv->ioaddr = addr; |