diff options
| author | Jon Mason <jon.mason@intel.com> | 2013-11-23 00:44:13 +0100 |
|---|---|---|
| committer | Jon Mason <jon.mason@intel.com> | 2014-04-07 19:58:14 +0200 |
| commit | fea903ecd799ec49a2ebc4cc8cfb7bb96fd31ddc (patch) | |
| tree | bfe894438f7c9a22b0ce1735106748d590cfdf0e /drivers/net/ntb_netdev.c | |
| parent | Linux 3.14 (diff) | |
| download | linux-fea903ecd799ec49a2ebc4cc8cfb7bb96fd31ddc.tar.xz linux-fea903ecd799ec49a2ebc4cc8cfb7bb96fd31ddc.zip | |
ntb_netdev: Fix list_for_each_entry exit issue
If list_for_each_entry exits without finding the ntb_device, the dev
pointer will not be NULL. Thus the check will never be true and the
code will not exit when it should. Correct this by adding a bool to
determine when the device is found, otherwise exit in good fashion.
Signed-off-by: Jon Mason <jon.mason@intel.com>
Diffstat (limited to 'drivers/net/ntb_netdev.c')
| -rw-r--r-- | drivers/net/ntb_netdev.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/net/ntb_netdev.c b/drivers/net/ntb_netdev.c index f3cdf64997d6..8298880bfdd8 100644 --- a/drivers/net/ntb_netdev.c +++ b/drivers/net/ntb_netdev.c @@ -367,12 +367,15 @@ static void ntb_netdev_remove(struct pci_dev *pdev) { struct net_device *ndev; struct ntb_netdev *dev; + bool found = false; list_for_each_entry(dev, &dev_list, list) { - if (dev->pdev == pdev) + if (dev->pdev == pdev) { + found = true; break; + } } - if (dev == NULL) + if (!found) return; list_del(&dev->list); |