From 12602d2a1f687a3e1e545489323ea6ddf778699b Mon Sep 17 00:00:00 2001 From: Michael Munday Date: Thu, 29 Dec 2016 16:50:58 -0500 Subject: [PATCH] Fix for DynASM buffer overflow. Need to include all actions with arguments against MAXSECPOS. --- dynasm/dasm_s390x.lua | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dynasm/dasm_s390x.lua b/dynasm/dasm_s390x.lua index f8c45fa8..9b9d3f4c 100644 --- a/dynasm/dasm_s390x.lua +++ b/dynasm/dasm_s390x.lua @@ -117,7 +117,7 @@ local function waction(action, val, a, num) wputxhw(w) if val then wputxhw(val) end -- Not sure about this, do we always have one arg? if a then actargs[#actargs+1] = a end - if a or num then secpos = secpos + (num or 1) end + if val or a or num then secpos = secpos + (num or 1) end end -- Flush action list (intervening C code or buffer pos overflow).