diff options
author | Valentin Bartenev <vbart@nginx.com> | 2012-12-23 15:36:52 +0000 |
---|---|---|
committer | Valentin Bartenev <vbart@nginx.com> | 2012-12-23 15:36:52 +0000 |
commit | a8ffed57510bab2a7143eb81aa3565a4bdcb8678 (patch) | |
tree | 301479cba5e394fb9a5165f4274a037c56ba56b5 /src/core/ngx_conf_file.c | |
parent | df71cd1125390512bbd421b0218b4bbc0e1d5209 (diff) | |
download | nginx-a8ffed57510bab2a7143eb81aa3565a4bdcb8678.tar.gz nginx-a8ffed57510bab2a7143eb81aa3565a4bdcb8678.zip |
Reopening log files code moved to a separate function.
The code refactored in a way to call custom handler that can do appropriate
cleanup work (if any), like flushing buffers, finishing compress streams,
finalizing connections to log daemon, etc..
Diffstat (limited to 'src/core/ngx_conf_file.c')
-rw-r--r-- | src/core/ngx_conf_file.c | 22 |
1 files changed, 3 insertions, 19 deletions
diff --git a/src/core/ngx_conf_file.c b/src/core/ngx_conf_file.c index 12bb5cf8a..0eb6388cb 100644 --- a/src/core/ngx_conf_file.c +++ b/src/core/ngx_conf_file.c @@ -945,7 +945,7 @@ ngx_conf_open_file(ngx_cycle_t *cycle, ngx_str_t *name) file->name = *name; } - file->buffer = NULL; + file->flush = NULL; return file; } @@ -954,7 +954,6 @@ ngx_conf_open_file(ngx_cycle_t *cycle, ngx_str_t *name) static void ngx_conf_flush_files(ngx_cycle_t *cycle) { - ssize_t n, len; ngx_uint_t i; ngx_list_part_t *part; ngx_open_file_t *file; @@ -975,23 +974,8 @@ ngx_conf_flush_files(ngx_cycle_t *cycle) i = 0; } - len = file[i].pos - file[i].buffer; - - if (file[i].buffer == NULL || len == 0) { - continue; - } - - n = ngx_write_fd(file[i].fd, file[i].buffer, len); - - if (n == -1) { - ngx_log_error(NGX_LOG_ALERT, cycle->log, ngx_errno, - ngx_write_fd_n " to \"%s\" failed", - file[i].name.data); - - } else if (n != len) { - ngx_log_error(NGX_LOG_ALERT, cycle->log, 0, - ngx_write_fd_n " to \"%s\" was incomplete: %z of %uz", - file[i].name.data, n, len); + if (file[i].flush) { + file[i].flush(&file[i], cycle->log); } } } |