Commitd434a6c

Chaim Sanders

authored and

Felipe Zimmerle

committed

Fixing missing return value check for hashing response injection failure

1 parentb3f197d commitd434a6cCopy full SHA for d434a6c

File tree

-2

lines changed

-2

lines changed

Lines changed: 6 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -609,8 +609,12 @@ static int flatten_response_body(modsec_rec *msr) {`
`609`	`609`	`retval=hash_response_body_links(msr);`
`610`	`610`	`if(retval>0) {`
`611`	`611`	`retval=inject_hashed_response_body(msr,retval);`
`612`		`-if (msr->txcfg->debuglog_level >=4) {`
`613`		`-msr_log(msr,4,"Hash completed in %"APR_TIME_T_FMT" usec.", (apr_time_now()-time1));`
	`612`	`+if(retval<0){`
	`613`	`+msr_log(msr,1,"inject_hashed_response_body: Unable to inject hash into response body. Returning response without changes." );`
	`614`	`+ }else{`
	`615`	`+if (msr->txcfg->debuglog_level >=4) {`
	`616`	`+msr_log(msr,4,"Hash completed in %"APR_TIME_T_FMT" usec.", (apr_time_now()-time1));`
	`617`	`+ }`
`614`	`618`	`}`
`615`	`619`
`616`	`620`	`}`

Comments

(0)