tree 4084f021969672141b4921566cc9f80df606d78d
parent 70ab149fe9c64b8173688042944932aa6615b1e0
author Aurelien DARRAGON <adarragon@haproxy.com> 1691568709 +0200
committer Christopher Faulet <cfaulet@haproxy.com> 1696414781 +0200

BUG/MINOR: hlua: fix invalid use of lua_pop on error paths

Multiple error paths made invalid use of lua_pop():

When the stack is emptied using lua_settop(0), lua_pop() (which is
implemented as a lua_settop() macro) should not be used right after,
because it could lead to invalid reads since the stack is already empty.

Unfortunately, some remnants from initial lua stack implementation kept
doing so, resulting in haproxy crashs on some lua runtime errors paths
from time to time (ie: ERRRUN, ERRMEM).

Moreover, the extra lua_pop() instruction, even if it was safe, is totally
pointless in such case.

Removing such unsafe lua_pop() statements when we know that the stack is
already empty.

This must be backported in every stable versions.

(cherry picked from commit 12cf8d4db7f0d239caf7ff3d94b62da2d953495c)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit 3e81aed0ad8373435d656266ea6a46dc49c15977)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit 2872047eb879de5c87324fe7d717468d592502d0)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit 931aaa31e8bd2c2b085cdf3c8e9c6aac61ff4941)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>