github_mirrors/ModSecurity
3
0
Fork 1
mirror of https://github.com/owasp-modsecurity/ModSecurity.git synced 2025-08-14 05:45:59 +03:00
Code Issues Packages Projects Releases Wiki Activity

Fix memory leak of RuleMessages objects

Browse Source
This commit is contained in:
martinhsv 2020-11-08 10:31:38 -08:00 committed by Felipe Zimmerle
parent 0466a5ab57
commit f8740e1144
No known key found for this signature in database
GPG Key ID: E6DFB08CE8B11277
3 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
CHANGES
View File

@ -1,6 +1,8 @@
v3.x.y - YYYY-MMM-DD (to be released)
-------------------------------------
- Fix memory leak of RuleMessages objects
[#2376 - @WGH-, @martinhsv]
- Produce not-supported error for ctl:forceRequestBodyVariable
and ctl:auditEngine
[#2376 - @WGH-, @martinhsv]

2
headers/modsecurity/transaction.h
View File

@ -343,7 +343,7 @@ class TransactionRuleMessageManagement {
* of the actions: `log_data' and `msg'. These should be included on the
* auditlogs.
*/
std::list<RuleMessage *> m_rulesMessages;
std::list<std::shared_ptr<RuleMessage>> m_rulesMessages;
Transaction *m_transaction;
};

10
src/transaction.cc
View File

@ -64,11 +64,11 @@ namespace modsecurity {
RuleMessage *TransactionRuleMessageManagement::messageGetLast() {
return m_rulesMessages.back();
return m_rulesMessages.back().get();
}
void TransactionRuleMessageManagement::logMatchLastRuleOnTheChain(const RuleWithActions *rule) {
RuleMessage *rm = m_rulesMessages.back();
RuleMessage *rm = m_rulesMessages.back().get();
rm->setRule(rule);
@ -85,13 +85,13 @@ void TransactionRuleMessageManagement::logMatchLastRuleOnTheChain(const RuleWith
}
void TransactionRuleMessageManagement::messageNew() {
m_rulesMessages.push_back(new RuleMessage(m_transaction));
m_rulesMessages.push_back(std::make_shared<RuleMessage>(m_transaction));
}
std::list<RuleMessage *> TransactionRuleMessageManagement::messageGetAll() {
std::list<RuleMessage *> messages;
for (RuleMessage *a : m_rulesMessages) {
messages.push_back(a);
for (auto iter = m_rulesMessages.begin(); iter != m_rulesMessages.end(); ++iter) {
messages.push_back((*iter).get());
}
return messages;
}