ngx_write_chain_to_file cannot be closed file when write error

[yugo-horie@…]

ngx_write_chain_to_file cannot be closed file when ngx_write_file returns NGX_ERROR.
It seems to be a same issue below. I'm not sure why here is to be missing file closing.
​https://github.com/nginx/nginx/pull/52/
​http://hg.nginx.org/nginx/rev/4f30f75dbdf3

patch

diff --git a/src/os/unix/ngx_files.c b/src/os/unix/ngx_files.c
index 1c82a8ea..7d3d5be8 100644
--- a/src/os/unix/ngx_files.c
+++ b/src/os/unix/ngx_files.c
@@ -300,9 +300,13 @@ ngx_write_chain_to_file(ngx_file_t *file, ngx_chain_t *cl, off_t offset,
     /* use pwrite() if there is the only buf in a chain */

     if (cl->next == NULL) {
-        return ngx_write_file(file, cl->buf->pos,
+        n = ngx_write_file(file, cl->buf->pos,
                               (size_t) (cl->buf->last - cl->buf->pos),
                               offset);
+        if (n == NGX_ERROR) {
+            (void) ngx_close_file(file->fd);
+        }
+        return n;
     }

     total = 0;
@@ -321,6 +325,7 @@ ngx_write_chain_to_file(ngx_file_t *file, ngx_chain_t *cl, off_t offset,
                                iovs[0].iov_len, offset);

             if (n == NGX_ERROR) {
+                (void) ngx_close_file(file->fd);
                 return n;
             }

@@ -330,6 +335,7 @@ ngx_write_chain_to_file(ngx_file_t *file, ngx_chain_t *cl, o
ff_t offset,
         n = ngx_writev_file(file, &vec, offset);

         if (n == NGX_ERROR) {
+            (void) ngx_close_file(file->fd);
             return n;
         }

[Maxim Dounin]

Since ngx_write_chain_to_file() does not open the file, it's not its responsibility to close it.

[yugo-horie@…]

Thanks comment.
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.
It might have another procedure which sweep and close file stream which remains opening, but I cannot find it.

[Maxim Dounin]

All files opened during request processing are bound to the request pool, and closed automatically once the request is freed.