[PATCH 05/15] batman-adv: use list_for_each_entry_safe

[Antonio Quartulli]

From: Geliang Tang <geliangt...@163.com>

Use list_for_each_entry_safe() instead of list_for_each_safe() to
simplify the code.

Signed-off-by: Geliang Tang <geliangt...@163.com>
Acked-by: Antonio Quartulli <a...@unstable.cc>
Reviewed-by: Sven Eckelmann <s...@narfation.org>
Signed-off-by: Marek Lindner <mareklind...@neomailbox.ch>
Signed-off-by: Antonio Quartulli <a...@unstable.cc>
---
 net/batman-adv/icmp_socket.c | 22 +++++++++-------------
 1 file changed, 9 insertions(+), 13 deletions(-)

diff --git a/net/batman-adv/icmp_socket.c b/net/batman-adv/icmp_socket.c
index 14d0013b387e..8a5889d134bc 100644
--- a/net/batman-adv/icmp_socket.c
+++ b/net/batman-adv/icmp_socket.c
@@ -104,25 +104,21 @@ static int batadv_socket_open(struct inode *inode, struct 
file *file)
 
 static int batadv_socket_release(struct inode *inode, struct file *file)
 {
-       struct batadv_socket_client *socket_client = file->private_data;
-       struct batadv_socket_packet *socket_packet;
-       struct list_head *list_pos, *list_pos_tmp;
+       struct batadv_socket_client *client = file->private_data;
+       struct batadv_socket_packet *packet, *tmp;
 
-       spin_lock_bh(&socket_client->lock);
+       spin_lock_bh(&client->lock);
 
        /* for all packets in the queue ... */
-       list_for_each_safe(list_pos, list_pos_tmp, &socket_client->queue_list) {
-               socket_packet = list_entry(list_pos,
-                                          struct batadv_socket_packet, list);
-
-               list_del(list_pos);
-               kfree(socket_packet);
+       list_for_each_entry_safe(packet, tmp, &client->queue_list, list) {
+               list_del(&packet->list);
+               kfree(packet);
        }
 
-       batadv_socket_client_hash[socket_client->index] = NULL;
-       spin_unlock_bh(&socket_client->lock);
+       batadv_socket_client_hash[client->index] = NULL;
+       spin_unlock_bh(&client->lock);
 
-       kfree(socket_client);
+       kfree(client);
        module_put(THIS_MODULE);
 
        return 0;
-- 
2.8.2