Changes between Initial Version and Version 1 of Ticket #2185, comment 2


Ignore:
Timestamp:
05/18/21 06:21:26 (3 years ago)
Author:
yugo-horie@…

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #2185, comment 2

    initial v1  
    11Thanks comment.
    2 In case ngx_write_chain_to_file has been called by `ngx_event_pipe` which returns NGX_ABORT for failing `ngx_write_file`(in this case, `ngx_event_pipe` is only called from ngx_http_upstream, all of them returns `ngx_http_upstream_finalize_request(r, u, NGX_ERROR);` ), I considered that it should be closed the file stream to prevent from resource leaks.
     2In case `ngx_write_chain_to_file` has been called by `ngx_event_pipe` which returns NGX_ABORT for failing `ngx_write_file`(in this case, `ngx_event_pipe` is only called from ngx_http_upstream, all of them return `ngx_http_upstream_finalize_request(r, u, NGX_ERROR);` ), I considered that it should be closed the file stream to prevent from resource leaks.
    33It might have another procedure which sweep and close file stream which remains opening, but I cannot find it.