Using malloc/free for the http fields hash.

This is required due to lack of a graceful shutdown: there is a small gap
between the runtime's memory pool release and router process's exit. Thus, a
worker thread may start processing a request between these two operations,
which may result in an http fields hash access and subsequent crash.

To simplify issue reproduction, it makes sense to add a 2 sec sleep before
exit() in nxt_runtime_exit().

1 files changed, 3 insertions, 3 deletions

diff --git a/src/nxt_http_request.c b/src/nxt_http_request.c
index 72aaa290..050587f7 100644
--- a/src/nxt_http_request.c
+++ b/src/nxt_http_request.c
@@ -36,17 +36,17 @@ nxt_time_string_t  nxt_http_date_cache = {
 
 
 nxt_int_t
-nxt_http_init(nxt_task_t *task, nxt_runtime_t *rt)
+nxt_http_init(nxt_task_t *task)
 {
     nxt_int_t  ret;
 
-    ret = nxt_h1p_init(task, rt);
+    ret = nxt_h1p_init(task);
 
     if (ret != NXT_OK) {
         return ret;
     }
 
-    return nxt_http_response_hash_init(task, rt);
+    return nxt_http_response_hash_init(task);
 }