Avoids copy on the transformation operation

headers/modsecurity/actions/action.h

@@ -57,7 +57,7 @@ class Action {
 
     virtual ~Action() { }
 
-    virtual std::string evaluate(std::string exp,
+    virtual std::string evaluate(const std::string &exp,
         Transaction *transaction);
     virtual bool evaluate(Rule *rule, Transaction *transaction);
     virtual bool evaluate(Rule *rule, Transaction *transaction,

src/actions/action.cc

@@ -45,7 +45,7 @@ namespace modsecurity {
 namespace actions {
 
 
-std::string Action::evaluate(std::string value,
+std::string Action::evaluate(const std::string &value,
     Transaction *transaction) {
     return value;
 }

src/actions/transformations/base64_decode.cc

@@ -32,7 +32,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string Base64Decode::evaluate(std::string value,
+std::string Base64Decode::evaluate(const std::string &value,
     Transaction *transaction) {
     std::string ret = Utils::Base64::decode(value);
 

src/actions/transformations/base64_decode.h

@@ -32,7 +32,7 @@ class Base64Decode : public Transformation {
  public:
     explicit Base64Decode(const std::string &action)  : Transformation(action) { }
 
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/base64_decode_ext.cc

@@ -32,7 +32,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string Base64DecodeExt::evaluate(std::string value,
+std::string Base64DecodeExt::evaluate(const std::string &value,
     Transaction *transaction) {
     std::string ret = Utils::Base64::decode_forgiven(value);
 

src/actions/transformations/base64_decode_ext.h

@@ -32,7 +32,7 @@ class Base64DecodeExt : public Transformation {
  public:
     explicit Base64DecodeExt(const std::string &action)  : Transformation(action) { }
 
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/base64_encode.cc

@@ -32,7 +32,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string Base64Encode::evaluate(std::string value,
+std::string Base64Encode::evaluate(const std::string &value,
     Transaction *transaction) {
     std::string ret = Utils::Base64::encode(value);
 

src/actions/transformations/base64_encode.h

@@ -32,7 +32,7 @@ class Base64Encode : public Transformation {
  public:
     explicit Base64Encode(const std::string &action)  : Transformation(action) { }
 
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/cmd_line.cc

@@ -31,7 +31,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string CmdLine::evaluate(std::string value,
+std::string CmdLine::evaluate(const std::string &value,
     Transaction *transaction) {
     std::string ret;
     int space = 0;

src/actions/transformations/cmd_line.h

@@ -33,7 +33,7 @@ class CmdLine : public Transformation {
     explicit CmdLine(const std::string &action) 
         : Transformation(action) { }
 
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/compress_whitespace.cc

@@ -35,7 +35,7 @@ CompressWhitespace::CompressWhitespace(const std::string &action)
     this->action_kind = 1;
 }
 
-std::string CompressWhitespace::evaluate(std::string value,
+std::string CompressWhitespace::evaluate(const std::string &value,
     Transaction *transaction) {
 
     std::string a;

src/actions/transformations/compress_whitespace.h

@@ -30,8 +30,10 @@ namespace transformations {
 
 class CompressWhitespace : public Transformation {
  public:
+
     explicit CompressWhitespace(const std::string &action) ;
-    std::string evaluate(std::string exp,
+
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/css_decode.cc

@@ -34,7 +34,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string CssDecode::evaluate(std::string value,
+std::string CssDecode::evaluate(const std::string &value,
     Transaction *transaction) {
 
     char *tmp = reinterpret_cast<char *>(

src/actions/transformations/css_decode.h

@@ -33,7 +33,7 @@ class CssDecode : public Transformation {
  public:
     explicit CssDecode(const std::string &action) 
         : Transformation(action) { }
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 
     static int css_decode_inplace(unsigned char *input, int64_t input_len);

src/actions/transformations/escape_seq_decode.cc

@@ -140,7 +140,7 @@ int EscapeSeqDecode::ansi_c_sequences_decode_inplace(unsigned char *input,
 }
 
 
-std::string EscapeSeqDecode::evaluate(std::string value,
+std::string EscapeSeqDecode::evaluate(const std::string &value,
     Transaction *transaction) {
 
     unsigned char *tmp = (unsigned char *) malloc(sizeof(char)

src/actions/transformations/escape_seq_decode.h

@@ -30,8 +30,10 @@ namespace transformations {
 
 class EscapeSeqDecode : public Transformation {
  public:
+
     explicit EscapeSeqDecode(const std::string &action) ;
-    std::string evaluate(std::string exp,
+
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
     int ansi_c_sequences_decode_inplace(unsigned char *input, int input_len);
 };

src/actions/transformations/hex_decode.cc

@@ -32,7 +32,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string HexDecode::evaluate(std::string value,
+std::string HexDecode::evaluate(const std::string &value,
     Transaction *transaction) {
     std::string ret;
     unsigned char *input;

src/actions/transformations/hex_decode.h

@@ -32,7 +32,7 @@ class HexDecode : public Transformation {
  public:
     explicit HexDecode(const std::string &action)  : Transformation(action) { }
 
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 
     static int inplace(unsigned char *data, int len);

src/actions/transformations/hex_encode.cc

@@ -36,7 +36,7 @@ HexEncode::HexEncode(const std::string &action)
     this->action_kind = 1;
 }
 
-std::string HexEncode::evaluate(std::string value,
+std::string HexEncode::evaluate(const std::string &value,
     Transaction *transaction) {
 
     std::stringstream result;

src/actions/transformations/hex_encode.h

@@ -30,8 +30,10 @@ namespace transformations {
 
 class HexEncode : public Transformation {
  public:
-    explicit HexEncode(const std::string &action) ;
+
-    std::string evaluate(std::string exp,
+    explicit HexEncode(const std::string &action);
+
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/html_entity_decode.cc

@@ -33,7 +33,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string HtmlEntityDecode::evaluate(std::string value,
+std::string HtmlEntityDecode::evaluate(const std::string &value,
     Transaction *transaction) {
     std::string ret;
     unsigned char *input;

src/actions/transformations/html_entity_decode.h

@@ -36,7 +36,7 @@ class HtmlEntityDecode : public Transformation {
     explicit HtmlEntityDecode(const std::string &action) 
         : Transformation(action) { }
 
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 
     static int inplace(unsigned char *input, uint64_t input_len);

src/actions/transformations/js_decode.cc

@@ -34,7 +34,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string JsDecode::evaluate(std::string value,
+std::string JsDecode::evaluate(const std::string &value,
     Transaction *transaction) {
     std::string ret;
     unsigned char *input;

src/actions/transformations/js_decode.h

@@ -33,7 +33,7 @@ class JsDecode : public Transformation {
     explicit JsDecode(const std::string &action) 
         : Transformation(action) { }
 
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
     static int inplace(unsigned char *input, uint64_t input_len);
 };

src/actions/transformations/length.cc

@@ -35,7 +35,7 @@ Length::Length(const std::string &action)
     this->action_kind = 1;
 }
 
-std::string Length::evaluate(std::string value,
+std::string Length::evaluate(const std::string &value,
     Transaction *transaction) {
 
     return std::to_string(value.size());

src/actions/transformations/length.h

@@ -30,8 +30,10 @@ namespace transformations {
 
 class Length : public Transformation {
  public:
-    explicit Length(const std::string &action) ;
+
-    std::string evaluate(std::string exp,
+    explicit Length(const std::string &action);
+
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/lower_case.cc

@@ -31,9 +31,10 @@ LowerCase::LowerCase(const std::string &a)
     : Transformation(a) {
 }
 
-std::string LowerCase::evaluate(std::string value,
+std::string LowerCase::evaluate(const std::string &val,
     Transaction *transaction) {
     std::locale loc;
+    std::string value(val);
 
     for (std::string::size_type i=0; i < value.length(); ++i) {
         value[i] = std::tolower(value[i], loc);

src/actions/transformations/lower_case.h

@@ -32,8 +32,8 @@ namespace transformations {
 
 class LowerCase : public Transformation {
  public:
-    explicit LowerCase(const std::string &action) ;
+    explicit LowerCase(const std::string &action);
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/md5.cc

@@ -31,9 +31,9 @@ namespace actions {
 namespace transformations {
 
 
-std::string Md5::evaluate(std::string value,
+std::string Md5::evaluate(const std::string &value,
     Transaction *transaction) {
-    std::string ret =  Utils::Md5::digest(value);
+    std::string ret = Utils::Md5::digest(value);
 
     return ret;
 }

src/actions/transformations/md5.h

@@ -32,7 +32,7 @@ class Md5 : public Transformation {
  public:
     explicit Md5(const std::string &action)  : Transformation(action) { }
 
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/none.cc

@@ -31,7 +31,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string None::evaluate(std::string value,
+std::string None::evaluate(const std::string &value,
     Transaction *transaction) {
     return value;
 }

src/actions/transformations/none.h

@@ -34,7 +34,7 @@ class None : public Transformation {
         : Transformation(action)
         { m_isNone = true; }
 
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/normalise_path.cc

@@ -37,7 +37,7 @@ NormalisePath::NormalisePath(const std::string &action)
     this->action_kind = 1;
 }
 
-std::string NormalisePath::evaluate(std::string value,
+std::string NormalisePath::evaluate(const std::string &value,
     Transaction *transaction) {
     int changed = 0;
 

src/actions/transformations/normalise_path.h

@@ -30,8 +30,10 @@ namespace transformations {
 
 class NormalisePath : public Transformation {
  public:
-    explicit NormalisePath(const std::string &action) ;
+
-    std::string evaluate(std::string exp,
+    explicit NormalisePath(const std::string &action);
+
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 
     static int normalize_path_inplace(unsigned char *input, int input_len,

src/actions/transformations/normalise_path_win.cc

@@ -34,7 +34,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string NormalisePathWin::evaluate(std::string value,
+std::string NormalisePathWin::evaluate(const std::string &value,
     Transaction *transaction) {
     int changed;
 

src/actions/transformations/normalise_path_win.h

@@ -33,7 +33,7 @@ class NormalisePathWin : public Transformation {
     explicit NormalisePathWin(const std::string &action) 
         : Transformation(action) { }
 
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/parity_even_7bit.cc

@@ -32,7 +32,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string ParityEven7bit::evaluate(std::string value,
+std::string ParityEven7bit::evaluate(const std::string &value,
     Transaction *transaction) {
     std::string ret;
     unsigned char *input;

src/actions/transformations/parity_even_7bit.h

@@ -32,7 +32,7 @@ class ParityEven7bit : public Transformation {
  public:
     explicit ParityEven7bit(const std::string &action)  : Transformation(action) { }
 
-    std::string evaluate(std::string exp, Transaction *transaction) override;
+    std::string evaluate(const std::string &exp, Transaction *transaction) override;
     static bool inplace(unsigned char *input, uint64_t input_len);
 };
 

src/actions/transformations/parity_odd_7bit.cc

@@ -32,7 +32,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string ParityOdd7bit::evaluate(std::string value,
+std::string ParityOdd7bit::evaluate(const std::string &value,
     Transaction *transaction) {
     std::string ret;
     unsigned char *input;

src/actions/transformations/parity_odd_7bit.h

@@ -32,7 +32,7 @@ class ParityOdd7bit : public Transformation {
  public:
     explicit ParityOdd7bit(const std::string &action)  : Transformation(action) { }
 
-    std::string evaluate(std::string exp, Transaction *transaction) override;
+    std::string evaluate(const std::string &exp, Transaction *transaction) override;
     static bool inplace(unsigned char *input, uint64_t input_len);
 };
 

src/actions/transformations/parity_zero_7bit.cc

@@ -32,7 +32,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string ParityZero7bit::evaluate(std::string value,
+std::string ParityZero7bit::evaluate(const std::string &value,
     Transaction *transaction) {
     std::string ret;
     unsigned char *input;

src/actions/transformations/parity_zero_7bit.h

@@ -32,7 +32,7 @@ class ParityZero7bit : public Transformation {
  public:
     explicit ParityZero7bit(const std::string &action)  : Transformation(action) { }
 
-    std::string evaluate(std::string exp, Transaction *transaction) override;
+    std::string evaluate(const std::string &exp, Transaction *transaction) override;
     static bool inplace(unsigned char *input, uint64_t input_len);
 };
 

src/actions/transformations/remove_comments.cc

@@ -32,7 +32,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string RemoveComments::evaluate(std::string value,
+std::string RemoveComments::evaluate(const std::string &value,
     Transaction *transaction) {
     std::string ret;
     unsigned char *input;

src/actions/transformations/remove_comments.h

@@ -33,7 +33,7 @@ class RemoveComments : public Transformation {
  public:
     explicit RemoveComments(const std::string &action)  : Transformation(action) { }
 
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/remove_comments_char.cc

@@ -35,9 +35,10 @@ RemoveCommentsChar::RemoveCommentsChar(const std::string &action)
     this->action_kind = 1;
 }
 
-std::string RemoveCommentsChar::evaluate(std::string value,
+std::string RemoveCommentsChar::evaluate(const std::string &val,
     Transaction *transaction) {
     int64_t i;
+    std::string value(val);
 
     i = 0;
     while (i < value.size()) {

src/actions/transformations/remove_comments_char.h

@@ -30,8 +30,9 @@ namespace transformations {
 
 class RemoveCommentsChar : public Transformation {
  public:
-    explicit RemoveCommentsChar(const std::string &action) ;
+    explicit RemoveCommentsChar(const std::string &action);
-    std::string evaluate(std::string exp,
+
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/remove_nulls.cc

@@ -33,9 +33,10 @@ namespace actions {
 namespace transformations {
 
 
-std::string RemoveNulls::evaluate(std::string value,
+std::string RemoveNulls::evaluate(const std::string &val,
     Transaction *transaction) {
     int64_t i;
+    std::string value(val);
 
     i = 0;
     while (i < value.size()) {

src/actions/transformations/remove_nulls.h

@@ -33,7 +33,7 @@ class RemoveNulls : public Transformation {
     explicit RemoveNulls(const std::string &action) 
         : Transformation(action) { }
 
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/remove_whitespace.cc

@@ -36,8 +36,9 @@ RemoveWhitespace::RemoveWhitespace(const std::string &action)
     this->action_kind = 1;
 }
 
-std::string RemoveWhitespace::evaluate(std::string value,
+std::string RemoveWhitespace::evaluate(const std::string &val,
     Transaction *transaction) {
+    std::string value(val);
 
     int64_t i = 0;
 

src/actions/transformations/remove_whitespace.h

@@ -30,8 +30,9 @@ namespace transformations {
 
 class RemoveWhitespace : public Transformation {
  public:
-    explicit RemoveWhitespace(const std::string &action) ;
+    explicit RemoveWhitespace(const std::string &action);
-    std::string evaluate(std::string exp,
+
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/replace_comments.cc

@@ -36,7 +36,7 @@ ReplaceComments::ReplaceComments(const std::string &action)
     this->action_kind = 1;
 }
 
-std::string ReplaceComments::evaluate(std::string value,
+std::string ReplaceComments::evaluate(const std::string &value,
     Transaction *transaction) {
     uint64_t i, j, incomment;
 

src/actions/transformations/replace_comments.h

@@ -30,8 +30,10 @@ namespace transformations {
 
 class ReplaceComments : public Transformation {
  public:
+
     explicit ReplaceComments(const std::string &action) ;
-    std::string evaluate(std::string exp,
+
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/replace_nulls.cc

@@ -35,9 +35,10 @@ ReplaceNulls::ReplaceNulls(const std::string &action)
     this->action_kind = 1;
 }
 
-std::string ReplaceNulls::evaluate(std::string value,
+std::string ReplaceNulls::evaluate(const std::string &val,
     Transaction *transaction) {
     int64_t i;
+    std::string value(val);
 
     i = 0;
     while (i < value.size()) {

src/actions/transformations/replace_nulls.h

@@ -30,8 +30,10 @@ namespace transformations {
 
 class ReplaceNulls : public Transformation {
  public:
+
     explicit ReplaceNulls(const std::string &action) ;
-    std::string evaluate(std::string exp,
+
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/sha1.cc

@@ -36,7 +36,7 @@ Sha1::Sha1(const std::string &action)
     this->action_kind = 1;
 }
 
-std::string Sha1::evaluate(std::string value,
+std::string Sha1::evaluate(const std::string &value,
     Transaction *transaction) {
 
     return Utils::Sha1::digest(value);

src/actions/transformations/sha1.h

@@ -31,7 +31,7 @@ namespace transformations {
 class Sha1 : public Transformation {
  public:
     explicit Sha1(const std::string &action) ;
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/sql_hex_decode.cc

@@ -41,7 +41,7 @@ namespace transformations {
 #define ISODIGIT(X) ((X >= '0') && (X <= '7'))
 #endif
 
-std::string SqlHexDecode::evaluate(std::string value,
+std::string SqlHexDecode::evaluate(const std::string &value,
     Transaction *transaction) {
     std::string ret;
     unsigned char *input;

src/actions/transformations/sql_hex_decode.h

@@ -32,7 +32,7 @@ class SqlHexDecode : public Transformation {
  public:
     explicit SqlHexDecode(const std::string &action)  : Transformation(action) { }
 
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 
     static int inplace(unsigned char *data, int len);

src/actions/transformations/transformation.cc

@@ -69,7 +69,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string Transformation::evaluate(std::string value,
+std::string Transformation::evaluate(const std::string &value,
     Transaction *transaction) {
     return value;
 }

src/actions/transformations/transformation.h

@@ -35,7 +35,7 @@ class Transformation : public Action {
     explicit Transformation(const std::string& _action, int kind)
         : Action(_action, kind) { }
 
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 
     static Transformation* instantiate(std::string a);

src/actions/transformations/trim.cc

@@ -57,8 +57,9 @@ Trim::Trim(const std::string &action)
 
 
 std::string
-Trim::evaluate(std::string value,
+Trim::evaluate(const std::string &val,
     Transaction *transaction) {
+    std::string value(val);
     return *this->trim(&value);
 }
 

src/actions/transformations/trim.h

@@ -30,8 +30,10 @@ namespace transformations {
 
 class Trim : public Transformation {
  public:
+
     explicit Trim(const std::string &action) ;
-    std::string evaluate(std::string exp,
+
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 
     std::string *ltrim(std::string *s);

src/actions/transformations/trim_left.cc

@@ -38,8 +38,9 @@ TrimLeft::TrimLeft(const std::string &action)
     this->action_kind = 1;
 }
 
-std::string TrimLeft::evaluate(std::string value,
+std::string TrimLeft::evaluate(const std::string &val,
     Transaction *transaction) {
+        std::string value(val);
     return *ltrim(&value);
 }
 

src/actions/transformations/trim_left.h

@@ -32,7 +32,8 @@ namespace transformations {
 class TrimLeft : public Trim {
  public:
     explicit TrimLeft(const std::string &action) ;
-    std::string evaluate(std::string exp,
+
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/trim_right.cc

@@ -36,8 +36,9 @@ TrimRight::TrimRight(const std::string &action)
     this->action_kind = 1;
 }
 
-std::string TrimRight::evaluate(std::string value,
+std::string TrimRight::evaluate(const std::string &val,
     Transaction *transaction) {
+    std::string value(val);
     return *this->rtrim(&value);
 }
 

src/actions/transformations/trim_right.h

@@ -32,7 +32,8 @@ namespace transformations {
 class TrimRight : public Trim {
  public:
     explicit TrimRight(const std::string &action) ;
-    std::string evaluate(std::string exp,
+
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/upper_case.cc

@@ -31,8 +31,9 @@ UpperCase::UpperCase(const std::string &a)
     : Transformation(a) {
 }
 
-std::string UpperCase::evaluate(std::string value,
+std::string UpperCase::evaluate(const std::string &val,
     Transaction *transaction) {
+    std::string value(val);
     std::locale loc;
 
     for (std::string::size_type i=0; i < value.length(); ++i) {

src/actions/transformations/upper_case.h

@@ -33,7 +33,8 @@ namespace transformations {
 class UpperCase : public Transformation {
  public:
     explicit UpperCase(const std::string &action) ;
-    std::string evaluate(std::string exp,
+
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/url_decode.cc

@@ -37,7 +37,7 @@ UrlDecode::UrlDecode(const std::string &action)
     this->action_kind = 1;
 }
 
-std::string UrlDecode::evaluate(std::string value,
+std::string UrlDecode::evaluate(const std::string &value,
     Transaction *transaction) {
     unsigned char *val = NULL;
     int invalid_count = 0;

src/actions/transformations/url_decode.h

@@ -32,8 +32,10 @@ namespace transformations {
 
 class UrlDecode : public Transformation {
  public:
+
     explicit UrlDecode(const std::string &action) ;
-    std::string evaluate(std::string exp,
+
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 };
 

src/actions/transformations/url_decode_uni.cc

@@ -38,7 +38,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string UrlDecodeUni::evaluate(std::string value,
+std::string UrlDecodeUni::evaluate(const std::string &value,
     Transaction *t) {
     std::string ret;
     unsigned char *input;

src/actions/transformations/url_decode_uni.h

@@ -33,7 +33,7 @@ class UrlDecodeUni : public Transformation {
  public:
     explicit UrlDecodeUni(const std::string &action)  : Transformation(action) { }
 
-    std::string evaluate(std::string exp, Transaction *transaction) override;
+    std::string evaluate(const std::string &exp, Transaction *transaction) override;
     static int inplace(unsigned char *input, uint64_t input_len,
         Transaction *transaction);
 };

src/actions/transformations/url_encode.cc

@@ -87,7 +87,7 @@ std::string UrlEncode::url_enc(const char *input,
 }
 
 
-std::string UrlEncode::evaluate(std::string value,
+std::string UrlEncode::evaluate(const std::string &value,
     Transaction *transaction) {
     int changed;
 

src/actions/transformations/url_encode.h

@@ -30,8 +30,10 @@ namespace transformations {
 
 class UrlEncode : public Transformation {
  public:
+
     explicit UrlEncode(const std::string &action) ;
-    std::string evaluate(std::string exp,
+
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 
     std::string url_enc(const char *input,

src/actions/transformations/utf8_to_unicode.cc

@@ -33,7 +33,7 @@ namespace actions {
 namespace transformations {
 
 
-std::string Utf8ToUnicode::evaluate(std::string value,
+std::string Utf8ToUnicode::evaluate(const std::string &value,
     Transaction *transaction) {
     std::string ret;
     unsigned char *input;

src/actions/transformations/utf8_to_unicode.h

@@ -37,7 +37,7 @@ class Utf8ToUnicode : public Transformation {
  public:
     explicit Utf8ToUnicode(const std::string &action)  : Transformation(action) { }
 
-    std::string evaluate(std::string exp,
+    std::string evaluate(const std::string &exp,
         Transaction *transaction) override;
 
     static char *inplace(unsigned char *input, uint64_t input_len,

src/utils/md5.cc

@@ -7,7 +7,7 @@ namespace modsecurity {
 namespace Utils {
 
 
-std::string Md5::hexdigest(std::string& input) {
+std::string Md5::hexdigest(const std::string& input) {
     unsigned char digest[16];
 
     mbedtls_md5(reinterpret_cast<const unsigned char *>(input.c_str()),
@@ -22,7 +22,7 @@ std::string Md5::hexdigest(std::string& input) {
 }
 
 
-std::string Md5::digest(std::string& input) {
+std::string Md5::digest(const std::string& input) {
     unsigned char output[16];
     std::string ret;
 

src/utils/md5.h

@@ -31,8 +31,8 @@ class Md5 {
  public:
     Md5() { }
 
-    static std::string hexdigest(std::string& input);
+    static std::string hexdigest(const std::string& input);
-    static std::string digest(std::string& input);
+    static std::string digest(const std::string& input);
 };
 
 }  // namespace Utils