Fix: rculist header: use parenthesis around macro parameters

[urcu.git] / include / urcu / rculist.h

diff --git a/include/urcu/rculist.h b/include/urcu/rculist.h
index 1fd2df388f57cefc0ea19fa02a0c8c0b627847e8..753135e2cd98ab4981a7f0d73b2bc9ac72811a18 100644 (file)
--- a/include/urcu/rculist.h
+++ b/include/urcu/rculist.h
@@ -79,14 +79,13 @@ void cds_list_del_rcu(struct cds_list_head *elem)
 
 /* Iterate forward over the elements of the list.  */
 #define cds_list_for_each_rcu(pos, head) \
-       for (pos = rcu_dereference((head)->next); pos != (head); \
-               pos = rcu_dereference(pos->next))
-
+       for (pos = rcu_dereference((head)->next); (pos) != (head); \
+               pos = rcu_dereference((pos)->next))
 
 /* Iterate through elements of the list. */
 #define cds_list_for_each_entry_rcu(pos, head, member) \
-       for (pos = cds_list_entry(rcu_dereference((head)->next), __typeof__(*pos), member); \
-               &pos->member != (head); \
-               pos = cds_list_entry(rcu_dereference(pos->member.next), __typeof__(*pos), member))
+       for (pos = cds_list_entry(rcu_dereference((head)->next), __typeof__(*(pos)), member); \
+               &(pos)->member != (head); \
+               pos = cds_list_entry(rcu_dereference((pos)->member.next), __typeof__(*(pos)), member))
 
 #endif /* _URCU_RCULIST_H */