Merge pull request #2727 from liudongmiao/patch-1

fix memory leak when concurrent log includes REMOTE_USER

Author: martinhsv

src/transaction.cc

@@ -1508,6 +1508,9 @@ std::string Transaction::toOldAuditLogFormatIndex(const std::string &filename,
     variables::RemoteUser *r = new variables::RemoteUser("REMOTE_USER");
     std::vector<const VariableValue *> l;
     r->evaluate(this, NULL, &l);
+    for (auto &a : l) {
+        delete a;
+    }
     delete r;
 
     ss << utils::string::dash_if_empty(