Commit7e45982

Ewald Dieterich

authored and

Felipe Zimmerle

committed

Log why writing to audit log failed

Add error description from apr_strerror() to message that is logged ifwriting to audit log failed.

1 parent607dfd2 commit7e45982Copy full SHA for 7e45982

File tree

-2

lines changed

-2

lines changed

Lines changed: 5 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -49,10 +49,13 @@ static int sec_auditlog_write(modsec_rec msr, const char data, unsigned int le`
`49`	`49`	`/* Write data to file. */`
`50`	`50`	`rc=apr_file_write_full(msr->new_auditlog_fd,data,nbytes,&nbytes_written);`
`51`	`51`	`if (rc!=APR_SUCCESS) {`
	`52`	`+charerrstr[1024];`
	`53`	`+`
`52`	`54`	`msr_log(msr,1,"Audit log: Failed writing (requested %"APR_SIZE_T_FMT`
`53`		`-" bytes, written %"APR_SIZE_T_FMT")",nbytes,nbytes_written);`
	`55`	`+" bytes, written %"APR_SIZE_T_FMT"): %s",nbytes,nbytes_written,`
	`56`	`+apr_strerror(rc,errstr,sizeof(errstr));`
`54`	`57`
`55`		`-/* Concurrent log format:Don't leak file handle. */`
	`58`	`+/* Concurrent log format:don't leak file handle. */`
`56`	`59`	`if (msr->txcfg->auditlog_type==AUDITLOG_CONCURRENT) {`
`57`	`60`	`apr_file_close(msr->new_auditlog_fd);`
`58`	`61`	`}`

Comments

(0)