- activated testcase for set intersection, as it not works correctly

- some bugfixes for intersection
- corrected a logics bug in intersection

3 files changed, 17 insertions, 10 deletions

diff --git a/src/set/Makefile.am b/src/set/Makefile.am
index a2bbab9a6..a02d80b9e 100644
--- a/src/set/Makefile.am
+++ b/src/set/Makefile.am
@@ -66,7 +66,7 @@ libgnunetset_la_LDFLAGS = \
 
 if HAVE_TESTING
 check_PROGRAMS = \
- test_set_api test_set_union_result_full
+ test_set_api test_set_union_result_full test_set_intersection_result_full
 endif
 
 if ENABLE_TEST_RUN
@@ -92,6 +92,15 @@ test_set_union_result_full_LDADD = \
 test_set_union_result_full_DEPENDENCIES = \
   libgnunetset.la
 
+test_set_intersection_result_full_SOURCES = \
+ test_set_intersection_result_full.c
+test_set_intersection_result_full_LDADD = \
+  $(top_builddir)/src/util/libgnunetutil.la \
+  $(top_builddir)/src/testing/libgnunettesting.la \
+  $(top_builddir)/src/set/libgnunetset.la
+test_set_intersection_result_full_DEPENDENCIES = \
+  libgnunetset.la
+
 EXTRA_DIST = \
   test_set.conf
 
diff --git a/src/set/gnunet-service-set.c b/src/set/gnunet-service-set.c
index 3c089f395..73d8f95cd 100644
--- a/src/set/gnunet-service-set.c
+++ b/src/set/gnunet-service-set.c
@@ -728,8 +728,7 @@ handle_client_create_set (void *cls,
   switch (ntohs (msg->operation))
   {
   case GNUNET_SET_OPERATION_INTERSECTION:
-    // FIXME: implement intersection vt
-    // set->vt = _GSS_intersection_vt ();
+    set->vt = _GSS_intersection_vt ();
     break;
   case GNUNET_SET_OPERATION_UNION:
     set->vt = _GSS_union_vt ();
diff --git a/src/set/gnunet-service-set_intersection.c b/src/set/gnunet-service-set_intersection.c
index a28938e04..cc0c721af 100644
--- a/src/set/gnunet-service-set_intersection.c
+++ b/src/set/gnunet-service-set_intersection.c
@@ -210,8 +210,8 @@ iterator_initialization_by_alice (void *cls,
   struct GNUNET_HashCode mutated_hash;
 
   //only consider this element, if it is valid for us
-  if ((op->generation_created >= ee->generation_removed)
-       || (op->generation_created < ee->generation_added))
+  if ((op->generation_created < ee->generation_removed)
+       && (op->generation_created >= ee->generation_added))
     return GNUNET_YES;
 
   // not contained according to bob's bloomfilter
@@ -254,8 +254,8 @@ iterator_initialization (void *cls,
   struct Operation *op = cls;
 
   //only consider this element, if it is valid for us
-  if ((op->generation_created >= ee->generation_removed)
-       || (op->generation_created < ee->generation_added))
+  if ((op->generation_created < ee->generation_removed)
+       && (op->generation_created >= ee->generation_added))
     return GNUNET_YES;
 
   GNUNET_assert (GNUNET_YES ==
@@ -456,7 +456,7 @@ send_bloomfilter (struct Operation *op)
                                                 bf_elementbits);
 
   op->spec->salt++;
-  GNUNET_CONTAINER_multihashmap_iterate (op->spec->set->elements,
+  GNUNET_CONTAINER_multihashmap_iterate (op->state->my_elements,
                                          &iterator_bf_create,
                                          op);
 
@@ -602,7 +602,7 @@ process_bf (struct Operation *op){
   case PHASE_BF_EXCHANGE:
   case PHASE_MAYBE_FINISHED:
     // if we are bob or alice and are continuing operation
-    GNUNET_CONTAINER_multihashmap_iterate (op->spec->set->elements,
+    GNUNET_CONTAINER_multihashmap_iterate (op->state->my_elements,
                                            &iterator_bf_reduce,
                                            op);
     break;
@@ -919,7 +919,6 @@ static void
 intersection_add (struct SetState *set_state,
                   struct ElementEntry *ee)
 {
-  GNUNET_assert(0 < set_state->current_set_element_count);
   set_state->current_set_element_count++;
 }