After the loop ends, the iterator is not guaranteed to point to a valid
object and should not be used. Make it NULL to avoid undefined behavior.
Signed-off-by: Adrian Moreno <amore...@redhat.com>
---
include/openvswitch/list.h | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/include/openvswitch/list.h b/include/openvswitch/list.h
index 8673bab93..8cd76b892 100644
--- a/include/openvswitch/list.h
+++ b/include/openvswitch/list.h
@@ -112,8 +112,9 @@ static inline bool ovs_list_is_short(const struct ovs_list
*);
UPDATE_MULTIVAR_SAFE_LONG(VAR, NEXT))
#define LIST_FOR_EACH_POP(ITER, MEMBER, LIST) \
- while (!ovs_list_is_empty(LIST) \
- && (INIT_CONTAINER(ITER, ovs_list_pop_front(LIST), MEMBER), 1))
+ while (!ovs_list_is_empty(LIST) ? \
+ (INIT_CONTAINER(ITER, ovs_list_pop_front(LIST), MEMBER), 1) : \
+ (ITER = NULL, 0))
/* Inline implementations. */
--
2.34.1
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
