lib/filterx: fix scope cloning

The newly introduced `log_msg_has_changes` flag should not be copied as
the non-floating variables are not copied either.
This results in improperly tagging the clone as "tainted".

Also the generation counter should be set the previous value instead of 0 because
0 is used for signaling that the value should not be used.

Signed-off-by: Szilard Parrag <szilard.parrag@axoflow.com>

lib/filterx/filterx-scope.c

@@ -302,7 +302,7 @@ filterx_scope_clone(FilterXScope *other)
           g_array_append_val(self->variables, *v);
           FilterXVariable *v_clone = &g_array_index(self->variables, FilterXVariable, dst_index);
 
-          filterx_variable_set_generation(v_clone, 0);
+          filterx_variable_set_generation(v_clone, v->generation);
           if (v->value)
             v_clone->value = filterx_object_clone(v->value);
           else
@@ -316,7 +316,7 @@ filterx_scope_clone(FilterXScope *other)
   if (other->variables->len > 0)
     self->dirty = other->dirty;
   self->syncable = other->syncable;
-  self->log_msg_has_changes = other->log_msg_has_changes;
+
   msg_trace("Filterx clone finished",
             evt_tag_printf("scope", "%p", self),
             evt_tag_printf("other", "%p", other),