Commitb175c5c

littlecho

authored and

Felipe Zimmerle

committed

Update apache2_config.c

Change third parameter(which is the apr file permission flag) from CREATEMODE to dcfg->auditlog_fileperms. Due to the user can specify the desired file permission setting for the audit log files with setting the value of SecAuditLogFileMode, we should follow the file permission setting from the config file. Therefore, as the dcfg->auditlog_fileperms will be modified in cmd_audit_log_dirmode function, we can use the value while calling apr_file_open to meet the file permission that specified in modsecurity.conf.

1 parent35fbc76 commitb175c5cCopy full SHA for b175c5c

File tree

1 file changed

-2

lines changed

apache2
- apache2_config.c

1 file changed

-2

lines changed

`‎apache2/apache2_config.c‎`

Lines changed: 5 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -1192,10 +1192,13 @@ static const char cmd_audit_log(cmd_parms cmd, void _dcfg, const char p1)`
`1192`	`1192`	`else {`
`1193`	`1193`	`constchar*file_name=ap_server_root_relative(cmd->pool,dcfg->auditlog_name);`
`1194`	`1194`	`apr_status_trc;`
`1195`		`-`
	`1195`	`+`
	`1196`	`+if (dcfg->auditlog_fileperms==NOT_SET) {`
	`1197`	`+dcfg->auditlog_fileperms=CREATEMODE;`
	`1198`	`+ }`
`1196`	`1199`	`rc=apr_file_open(&dcfg->auditlog_fd,file_name,`
`1197`	`1200`	`APR_WRITE \|APR_APPEND \|APR_CREATE \|APR_BINARY,`
`1198`		`-CREATEMODE,cmd->pool);`
	`1201`	`+dcfg->auditlog_fileperms,cmd->pool);`
`1199`	`1202`
`1200`	`1203`	`if (rc!=APR_SUCCESS) {`
`1201`	`1204`	`returnapr_psprintf(cmd->pool,"ModSecurity: Failed to open the audit log file: %s",`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitb175c5c

File tree

1 file changed

1 file changed

`‎apache2/apache2_config.c‎`

0 commit comments