michael/lean2
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix bug reported by Valgrind. Reason: m_metavars is a vector of metavar_info; each metavar_info has a context; when we invoke mk_metavar the vector can grow in size, and the context is moved to a new location. The previous location is invalidated. To avoid the problem we have to save ctx in a local variable.

Browse source 
Signed-off-by: Leonardo de Moura <leonardo@microsoft.com>
This commit is contained in:
Leonardo de Moura 2013-09-04 04:28:09 -07:00
parent 9522048390
commit 57c0c69872
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/frontends/lean/lean_elaborator.cpp
View file

@ -121,7 +121,8 @@ class elaborator::imp {
if (m_metavars[midx].m_type) { if (m_metavars[midx].m_type) {
return m_metavars[midx].m_type; return m_metavars[midx].m_type;
} else { } else {
expr t = mk_metavar(m_metavars[midx].m_ctx); context ctx = m_metavars[midx].m_ctx;
expr t = mk_metavar(ctx);
m_metavars[midx].m_type = t; m_metavars[midx].m_type = t;
return t; return t;
} }