child-cfg: Fix removal of redundant traffic selectors

We have to make sure we compare every selected traffic selector with every
other in the list.

Fixes #577.

diff --git a/src/libcharon/config/child_cfg.c b/src/libcharon/config/child_cfg.c
index 6fe7d44b8fcf225313097911ace79ad468b2b5b0..8d8214ebe1aacaa7df0b7eab7b1ceba8b3304344 100644 (file)
--- a/src/libcharon/config/child_cfg.c
+++ b/src/libcharon/config/child_cfg.c
@@ -354,11 +354,11 @@ METHOD(child_cfg_t, get_traffic_selectors, linked_list_t*,
                                {
                                        result->remove_at(result, e1);
                                        ts1->destroy(ts1);
-                                       result->reset_enumerator(result, e2);
                                        break;
                                }
                        }
                }
+               result->reset_enumerator(result, e2);
        }
        e1->destroy(e1);
        e2->destroy(e2);