Skip to content

Commit 4b360b3

Browse files
committed
Fix memory leak in transaction rule removal lists
Valgrind reported memory leaks in m_ruleRemoveTargetByTag and related containers when ctl actions were used. This change explicitly clears m_ruleRemoveById, m_ruleRemoveByIdRange, m_ruleRemoveByTag, m_ruleRemoveTargetByTag, and m_ruleRemoveTargetById in the Transaction destructor to ensure all allocated memory is freed.
1 parent 36d8c2a commit 4b360b3

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

src/transaction.cc

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -170,6 +170,12 @@ Transaction::~Transaction() {
170170

171171
m_rulesMessages.clear();
172172

173+
m_ruleRemoveById.clear();
174+
m_ruleRemoveByIdRange.clear();
175+
m_ruleRemoveByTag.clear();
176+
m_ruleRemoveTargetById.clear();
177+
m_ruleRemoveTargetByTag.clear();
178+
173179
intervention::free(&m_it);
174180
intervention::clean(&m_it);
175181

0 commit comments

Comments
 (0)