staging: most: use unsafe version of list traversing
Christian Gromm
christian.gromm at microchip.com
Wed May 10 15:33:11 UTC 2017
From: Andrey Shvetsov <andrey.shvetsov at k2l.de>
The function get_net_dev_context does not remove elements of the list.
Hence, list traversing does not need to be secured.
This patch replaces list_for_each_entry_safe with the
list_for_each_entry.
Signed-off-by: Andrey Shvetsov <andrey.shvetsov at k2l.de>
Signed-off-by: Christian Gromm <christian.gromm at microchip.com>
---
drivers/staging/most/aim-network/networking.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/most/aim-network/networking.c b/drivers/staging/most/aim-network/networking.c
index a3009fdfa278..11f03d3cd565 100644
--- a/drivers/staging/most/aim-network/networking.c
+++ b/drivers/staging/most/aim-network/networking.c
@@ -290,11 +290,11 @@ static void most_net_rm_netdev_safe(struct net_dev_context *nd)
static struct net_dev_context *get_net_dev_context(
struct most_interface *iface)
{
- struct net_dev_context *nd, *tmp;
+ struct net_dev_context *nd;
unsigned long flags;
spin_lock_irqsave(&list_lock, flags);
- list_for_each_entry_safe(nd, tmp, &net_devices, list) {
+ list_for_each_entry(nd, &net_devices, list) {
if (nd->iface == iface) {
spin_unlock_irqrestore(&list_lock, flags);
return nd;
--
2.11.0
More information about the devel
mailing list