commit 42d1013ab9dff5efb466a7bb76054e29fafb69f0
author Maxim Dounin <mdounin@mdounin.ru> Thu Dec 05 19:38:06 2019 +0300
committer Maxim Dounin <mdounin@mdounin.ru> Thu Dec 05 19:38:06 2019 +0300
tree a66bccff532adaa29fcd325ca1705bf09ae15bf9
parent e3319a3aeacff4ce3a4825db5b61f906fc973a81

Upstream keepalive: clearing of c->data in cached connections.

Previously, connections returned from keepalive cache had c->data
pointing to the keepalive cache item.  While this shouldn't be a problem
for correct code, as c->data is not expected to be used before it is set,
explicitly clearing it might help to avoid confusion.

src/http/modules/ngx_http_upstream_keepalive_module.c

diff --git a/src/http/modules/ngx_http_upstream_keepalive_module.c b/src/http/modules/ngx_http_upstream_keepalive_module.c
index bdc4ae5..1560807 100644
--- a/src/http/modules/ngx_http_upstream_keepalive_module.c
+++ b/src/http/modules/ngx_http_upstream_keepalive_module.c
@@ -275,6 +275,7 @@
 
     c->idle = 0;
     c->sent = 0;
+    c->data = NULL;
     c->log = pc->log;
     c->read->log = pc->log;
     c->write->log = pc->log;