From e7b2e01d43d9c3ab6d223daa18eebb94a322e082 Mon Sep 17 00:00:00 2001 From: Roberto Ierusalimschy Date: Mon, 7 Mar 2016 16:25:39 -0300 Subject: [PATCH] bug: label between local definitions can mix-up their initializations --- bugs | 33 +++++++++++++++++++++++++++++++++ lparser.c | 4 ++-- 2 files changed, 35 insertions(+), 2 deletions(-) diff --git a/bugs b/bugs index 36eb87e5..38d295ae 100644 --- a/bugs +++ b/bugs @@ -3542,6 +3542,39 @@ patch = [[ } +Bug{ +what = [[label between local definitions can mix-up their initializations]], +report = [[Karel Tuma, 2016/03/01]], +since = [[5.2]], +fix = nil, +example = [[ +do + local k = 0 + local x + ::foo:: + local y -- should be reset to nil after goto, but it is not + assert(not y) + y = true + k = k + 1 + if k < 2 then goto foo end +end +]], +patch = [[ +--- lparser.c 2015/11/02 16:09:30 2.149 ++++ lparser.c 2016/03/03 12:03:37 +@@ -1226,7 +1226,7 @@ + checkrepeated(fs, ll, label); /* check for repeated labels */ + checknext(ls, TK_DBCOLON); /* skip double colon */ + /* create new entry for this label */ +- l = newlabelentry(ls, ll, label, line, fs->pc); ++ l = newlabelentry(ls, ll, label, line, luaK_getlabel(fs)); + skipnoopstat(ls); /* skip other no-op statements */ + if (block_follow(ls, 0)) { /* label is last no-op statement in the block? */ + /* assume that locals are already out of scope */ +]] +} + + --[=[ Bug{ what = [[ ]], diff --git a/lparser.c b/lparser.c index 757d6546..0b5d75a8 100644 --- a/lparser.c +++ b/lparser.c @@ -1,5 +1,5 @@ /* -** $Id: lparser.c,v 2.150 2015/12/09 15:21:28 roberto Exp roberto $ +** $Id: lparser.c,v 2.151 2016/01/05 16:22:37 roberto Exp roberto $ ** Lua Parser ** See Copyright Notice in lua.h */ @@ -1230,7 +1230,7 @@ static void labelstat (LexState *ls, TString *label, int line) { checkrepeated(fs, ll, label); /* check for repeated labels */ checknext(ls, TK_DBCOLON); /* skip double colon */ /* create new entry for this label */ - l = newlabelentry(ls, ll, label, line, fs->pc); + l = newlabelentry(ls, ll, label, line, luaK_getlabel(fs)); skipnoopstat(ls); /* skip other no-op statements */ if (block_follow(ls, 0)) { /* label is last no-op statement in the block? */ /* assume that locals are already out of scope */