commit 0fea0bf3f7982348fd6ec9d14ebd946c783445b0
author Sergey Kandaurov <pluknet@nginx.com> Tue Jul 30 15:04:46 2013 +0400
committer Sergey Kandaurov <pluknet@nginx.com> Tue Jul 30 15:04:46 2013 +0400
tree 185d65ce4f257a7b7c7ef57c0dcf2215f6562cd4
parent 2b2def789199b1fc661fbac38ff6bf7fdccb7bbb

Added safety belt for the case of sending header twice.

The aforementioned situation is abnormal per se and as such it now forces
request termination with appropriate error message.

src/http/ngx_http_core_module.c

diff --git a/src/http/ngx_http_core_module.c b/src/http/ngx_http_core_module.c
index ffe7fb4..47e5ff7 100644
--- a/src/http/ngx_http_core_module.c
+++ b/src/http/ngx_http_core_module.c
@@ -1933,6 +1933,12 @@
 ngx_int_t
 ngx_http_send_header(ngx_http_request_t *r)
 {
+    if (r->header_sent) {
+        ngx_log_error(NGX_LOG_ALERT, r->connection->log, 0,
+                      "header already sent");
+        return NGX_ERROR;
+    }
+
     if (r->err_status) {
         r->headers_out.status = r->err_status;
         r->headers_out.status_line.len = 0;