diff options
| author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2013-08-05 12:42:00 +0800 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2013-08-05 11:09:42 -0700 |
| commit | 1155e964a616a9bd4b603f6f8bf4f47576b010c5 (patch) | |
| tree | 1a5edcdee5cfd19660ce666192eb576878840730 /drivers | |
| parent | f094668ca1a3e326e4f3d75a5e3b32f949f417b8 (diff) | |
| download | linux-1155e964a616a9bd4b603f6f8bf4f47576b010c5.tar.bz2 | |
tile: fix missing unlock on error in tile_net_open()
Add the missing unlock before return from function tile_net_open()
in the error handling case.
Introduced by commit f3286a3af89d6db7a488f3e8f02b98d67d50f00c.
(tile: support multiple mPIPE shims in tilegx network driver)
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Acked-by: Chris Metcalf <cmetcalf@tilera.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/ethernet/tile/tilegx.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/net/ethernet/tile/tilegx.c b/drivers/net/ethernet/tile/tilegx.c index 907b5772fd55..5d2a719fc688 100644 --- a/drivers/net/ethernet/tile/tilegx.c +++ b/drivers/net/ethernet/tile/tilegx.c @@ -1510,8 +1510,10 @@ static int tile_net_open(struct net_device *dev) /* Get the instance info. */ rc = gxio_mpipe_link_instance(dev->name); - if (rc < 0 || rc >= NR_MPIPE_MAX) + if (rc < 0 || rc >= NR_MPIPE_MAX) { + mutex_unlock(&tile_net_devs_for_channel_mutex); return -EIO; + } priv->instance = rc; instance = rc; |