Fix memory issue while changing rule target dynamic

Issue #1590

Author: Felipe Zimmerle

CHANGES

@@ -2,6 +2,8 @@
 v3.0.????? - ?
 ---------------------------
 
+ - Fix memory issue while changing rule target dynamic
+   [Issue #1590 - @zimmerle, @slabber]
  - Fix log while displaying the name of a dict selection by regex.
    [@zimmerle]
  - Setting http response code on the auditlog.

src/rule.cc

@@ -447,10 +447,10 @@ std::vector<std::unique_ptr<collection::Variable>> Rule::getFinalVars(
             std::vector<const collection::Variable *> z;
             a.second->evaluateInternal(trans, this, &z);
             for (auto &y : z) {
-                exclusions_update_by_tag_remove.push_back(y->m_key);
+                exclusions_update_by_tag_remove.push_back(std::string(y->m_key));
                 delete y;
             }
-            exclusions_update_by_tag_remove.push_back(a.second->m_name);
+            exclusions_update_by_tag_remove.push_back(std::string(a.second->m_name));
 
         } else {
             Variable *b = a.second.get();
@@ -466,10 +466,10 @@ std::vector<std::unique_ptr<collection::Variable>> Rule::getFinalVars(
             std::vector<const collection::Variable *> z;
             a.second->evaluateInternal(trans, this, &z);
             for (auto &y : z) {
-                exclusions_update_by_id_remove.push_back(y->m_key);
+                exclusions_update_by_id_remove.push_back(std::string(y->m_key));
                 delete y;
             }
-            exclusions_update_by_id_remove.push_back(a.second->m_name);
+            exclusions_update_by_id_remove.push_back(std::string(a.second->m_name));
         } else {
             Variable *b = a.second.get();
             variables.push_back(b);
@@ -482,10 +482,10 @@ std::vector<std::unique_ptr<collection::Variable>> Rule::getFinalVars(
             std::vector<const collection::Variable *> z;
             variable->evaluateInternal(trans, this, &z);
             for (auto &y : z) {
-                exclusions.push_back(y->m_key);
+                exclusions.push_back(std::string(y->m_key));
                 delete y;
             }
-            exclusions.push_back(variable->m_name);
+            exclusions.push_back(std::string(variable->m_name));
         }
     }