github_mirrors/ModSecurity
3
0
Fork 1
mirror of https://github.com/owasp-modsecurity/ModSecurity.git synced 2025-11-16 01:22:18 +03:00
Code Issues Packages Projects Releases Wiki Activity

Removes copy form VariableValue

Browse Source 
On `Use std::shared_ptr for variable resolution` @WGH changes
VariableValue to be a shared_ptr. As shared pointer, the copy
on AnchoredVariable is no longer necessary. The copy was removed
along with the copy constructor.
This commit is contained in:
Felipe Zimmerle
2020-08-26 09:18:36 -03:00
parent 5a0e71fdc0
commit cc5d3f5106
3 changed files with 11 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
src/anchored_variable.cc
View File

@@ -34,7 +34,7 @@ AnchoredVariable::AnchoredVariable(Transaction *t,
m_offset(0),
m_name(name),
m_value(""),
m_var(&name) {
m_var(std::make_shared<VariableValue>(&name)) {
}
void AnchoredVariable::unset() {
@@ -50,7 +50,7 @@ void AnchoredVariable::set(const std::string &a, size_t offset,
VariableOrigin origin;
origin.m_offset = offset;
origin.m_length = offsetLen;
m_var.addOrigin(std::move(origin));
m_var->addOrigin(std::move(origin));
}
@@ -60,7 +60,7 @@ void AnchoredVariable::set(const std::string &a, size_t offset) {
VariableOrigin origin;
origin.m_offset = offset;
origin.m_length = m_value.size();
m_var.addOrigin(std::move(origin));
m_var->addOrigin(std::move(origin));
}
@@ -71,7 +71,7 @@ void AnchoredVariable::set(const char *a, size_t offset) {
m_value.assign(a, strlen(a));
origin.m_offset = offset;
origin.m_length = m_value.size();
m_var.addOrigin(std::move(origin));
m_var->addOrigin(std::move(origin));
}
@@ -83,7 +83,7 @@ void AnchoredVariable::set(const bpstd::string_view &a, size_t offset) {
origin.m_offset = offset;
origin.m_length = m_value.size();
m_var.addOrigin(std::move(origin));
m_var->addOrigin(std::move(origin));
}
@@ -98,7 +98,7 @@ void AnchoredVariable::append(const std::string &a, size_t offset,
VariableOrigin origin;
origin.m_offset = offset;
origin.m_length = a.size();
m_var.addOrigin(std::move(origin));
m_var->addOrigin(std::move(origin));
}
@@ -113,7 +113,7 @@ void AnchoredVariable::append(const std::string &a, size_t offset,
VariableOrigin origin;
origin.m_offset = offset;
origin.m_length = size;
m_var.addOrigin(std::move(origin));
m_var->addOrigin(std::move(origin));
}
@@ -122,8 +122,8 @@ void AnchoredVariable::evaluate(VariableValues *l) {
return;
}
m_var.setValue(m_value);
l->push_back(std::make_shared<VariableValue>(m_var));
m_var->setValue(m_value);
l->push_back(m_var);
}