Skip to content

Commit 7bdae25

Browse files
Fix -Wreorder in Transaction constructor
Initialize TransactionAnchoredVariables before m_logCbData in the member initializer list. Signed-off-by: Mikel Olasagasti Uranga <mikel@olasagasti.info>
1 parent 1ff0c45 commit 7bdae25

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

src/transaction.cc

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,8 @@ Transaction::Transaction(ModSecurity *ms, RulesSet *rules, const char *id, void
118118

119119
Transaction::Transaction(ModSecurity *ms, RulesSet *rules, const char *id,
120120
void *logCbData, const time_t timestamp)
121-
: m_creationTimeStamp(utils::cpu_seconds()),
121+
: TransactionAnchoredVariables(this),
122+
m_creationTimeStamp(utils::cpu_seconds()),
122123
m_ARGScombinedSizeDouble(0),
123124
m_clientPort(0),
124125
m_highestSeverityAction(255),
@@ -149,8 +150,7 @@ Transaction::Transaction(ModSecurity *ms, RulesSet *rules, const char *id,
149150
#endif
150151
m_secRuleEngine(RulesSetProperties::PropertyNotSetRuleEngine),
151152
m_secXMLParseXmlIntoArgs(rules->m_secXMLParseXmlIntoArgs),
152-
m_logCbData(logCbData),
153-
TransactionAnchoredVariables(this) {
153+
m_logCbData(logCbData) {
154154
m_variableUrlEncodedError.set("0", 0);
155155
m_variableMscPcreError.set("0", 0);
156156
m_variableMscPcreLimitsExceeded.set("0", 0);

0 commit comments

Comments
 (0)