commit 90df702bf86e4d9a9a2f319743598bc7aa1f8f57
author Maxim Dounin <mdounin@mdounin.ru> Mon Sep 08 21:36:03 2014 +0400
committer Maxim Dounin <mdounin@mdounin.ru> Mon Sep 08 21:36:03 2014 +0400
tree 20104d9186d929be93ce43240559241efdd429fc
parent c5bee22d167486537ed9f36941dc4e0abdef1fce

Fixed ETag memory allocation error handling.

The etag->hash must be set to 0 to avoid an empty ETag header being
returned with the 500 Internal Server Error page after the memory
allocation failure.

Reported by Markus Linnala.

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 4071b61..3b36f4a 100644
--- a/src/http/ngx_http_core_module.c
+++ b/src/http/ngx_http_core_module.c
@@ -1837,6 +1837,7 @@
 
     etag->value.data = ngx_pnalloc(r->pool, NGX_OFF_T_LEN + NGX_TIME_T_LEN + 3);
     if (etag->value.data == NULL) {
+        etag->hash = 0;
         return NGX_ERROR;
     }