Commit9ac9ff8

Felipe Zimmerle

committed

Adds a sanity check before use ctl:ruleRemoveTargetByTag

This commit closes the issueowasp-modsecurity#1353

1 parent112ba45 commit9ac9ff8Copy full SHA for 9ac9ff8

File tree

-0

lines changed

-0

lines changed

Lines changed: 2 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,8 @@`
`1`	`1`	`DD MMM YYYY - 2.9.2 - To be released`
`2`	`2`	`------------------------------------`
`3`	`3`
	`4`	`+ * Adds a sanity check before use ctl:ruleRemoveTargetByTag.`
	`5`	`+ [Issue #1353 - @LukeP21 and @zimmerle]`
`4`	`6`	`* Uses an optional global lock while manipulating collections.`
`5`	`7`	`[Issues #1224 - @mturk and @zimmerle]`
`6`	`8`	`* Fix collection naming problem while merging collections.`

Lines changed: 4 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -1253,6 +1253,10 @@ static apr_status_t msre_action_ctl_execute(modsec_rec msr, apr_pool_t mptmp,`
`1253`	`1253`	`if (msr->txcfg->debuglog_level >=4) {`
`1254`	`1254`	`msr_log(msr,4,"Ctl: ruleRemoveTargetByTag tag=%s targets=%s",p1,p2);`
`1255`	`1255`	`}`
	`1256`	`+if (p2==NULL) {`
	`1257`	`+msr_log(msr,1,"ModSecurity: Missing target for tag \"%s\"",p1);`
	`1258`	`+return-1;`
	`1259`	`+}`
`1256`	`1260`
`1257`	`1261`	`re=apr_pcalloc(msr->mp,sizeof(rule_exception));`
`1258`	`1262`	`re->type=RULE_EXCEPTION_REMOVE_TAG;`

Comments

(0)