From: Christopher Faulet <cfaulet@haproxy.com>
Date: Tue, 2 Jun 2020 16:46:07 +0000 (+0200)
Subject: BUG/MEDIUM: lua: Reset analyse expiration timeout before executing a lua action
X-Git-Tag: v2.2-dev9~158
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=23308ebf0d603d3f81b4c7a5bdcd82f1cd70c449;p=thirdparty%2Fhaproxy.git

BUG/MEDIUM: lua: Reset analyse expiration timeout before executing a lua action

Before executing a lua action, the analyse expiration timeout of the
corresponding channel must be reset. Otherwise, when it expires, for instance
because of a call to core.msleep(), if the action yields, an expired timeout
will be used for the stream's task, leading to a loop.

This patch should fix the issue #661. It must be backported in all versions
supporting the lua.
---

diff --git a/src/hlua.c b/src/hlua.c
index f0d7e0f9e6..f2b6620448 100644
--- a/src/hlua.c
+++ b/src/hlua.c
@@ -6618,6 +6618,15 @@ static enum act_return hlua_action(struct act_rule *rule, struct proxy *px,
 		s->hlua->max_time = hlua_timeout_session;
 	}
 
+	/* Always reset the analyse expiration timeout for the corresponding
+	 * channel in case the lua script yield, to be sure to not keep an
+	 * expired timeout.
+	 */
+	if (dir == SMP_OPT_DIR_REQ)
+		s->req.analyse_exp = TICK_ETERNITY;
+	else
+		s->res.analyse_exp = TICK_ETERNITY;
+
 	/* Execute the function. */
 	switch (hlua_ctx_resume(s->hlua, !(flags & ACT_OPT_FINAL))) {
 	/* finished. */