diff options
| author | Vasily Averin <vvs@virtuozzo.com> | 2020-06-02 14:55:26 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2020-06-03 00:35:53 +0200 |
| commit | e8224bfe77293494626f6eec1884fee7b87d0ced (patch) | |
| tree | d50e6f890a7b2cf8aae699bed32164342879bdfa /drivers/net/net_failover.c | |
| parent | Merge branch 'tipc-revert-two-patches' (diff) | |
| download | linux-e8224bfe77293494626f6eec1884fee7b87d0ced.tar.xz linux-e8224bfe77293494626f6eec1884fee7b87d0ced.zip | |
net_failover: fixed rollback in net_failover_open()
found by smatch:
drivers/net/net_failover.c:65 net_failover_open() error:
we previously assumed 'primary_dev' could be null (see line 43)
Fixes: cfc80d9a1163 ("net: Introduce net_failover driver")
Signed-off-by: Vasily Averin <vvs@virtuozzo.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
| -rw-r--r-- | drivers/net/net_failover.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/net/net_failover.c b/drivers/net/net_failover.c index b16a1221d19b..fb182bec8f06 100644 --- a/drivers/net/net_failover.c +++ b/drivers/net/net_failover.c @@ -61,7 +61,8 @@ static int net_failover_open(struct net_device *dev) return 0; err_standby_open: - dev_close(primary_dev); + if (primary_dev) + dev_close(primary_dev); err_primary_open: netif_tx_disable(dev); return err; |