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 6fe7d44..8d8214e 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->remove_at(result, e1);
                                        ts1->destroy(ts1);

-                                       result->reset_enumerator(result, e2);

                                        break;
                                }
                        }
                }
                                        break;
                                }
                        }
                }

+               result->reset_enumerator(result, e2);

        }
        e1->destroy(e1);
        e2->destroy(e2);
        }
        e1->destroy(e1);
        e2->destroy(e2);