summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndres Noetzli <andres.noetzli@gmail.com>2020-08-02 05:57:24 -0700
committerGitHub <noreply@github.com>2020-08-02 07:57:24 -0500
commit76b25e75c198f9af7ba2c2554e07fec5ba047597 (patch)
tree6f962fcebc54b9a65e181434333a7057e8b99fed
parent475985ccc80b1ddb38d912a3c6658912f1fc6207 (diff)
Fix ASan failure in interactive_shell_black (#4827)
This commit fixes an issue reported by ASan for unit test interactive_shell_black. The unit test was failing because nodes were created in the wrong node manager. The issue was likely introduced with e8bbee7.
-rw-r--r--src/api/cvc4cpp.cpp10
1 files changed, 8 insertions, 2 deletions
diff --git a/src/api/cvc4cpp.cpp b/src/api/cvc4cpp.cpp
index 1c15466a1..61d180fe4 100644
--- a/src/api/cvc4cpp.cpp
+++ b/src/api/cvc4cpp.cpp
@@ -1414,13 +1414,19 @@ size_t OpHashFunction::operator()(const Op& t) const
Term::Term() : d_solver(nullptr), d_node(new CVC4::Node()) {}
Term::Term(const Solver* slv, const CVC4::Expr& e)
- : d_solver(slv), d_node(new CVC4::Node(e))
+ : d_solver(slv)
{
+ // Ensure that we create the node in the correct node manager.
+ NodeManagerScope scope(d_solver->getNodeManager());
+ d_node.reset(new CVC4::Node(e));
}
Term::Term(const Solver* slv, const CVC4::Node& n)
- : d_solver(slv), d_node(new CVC4::Node(n))
+ : d_solver(slv)
{
+ // Ensure that we create the node in the correct node manager.
+ NodeManagerScope scope(d_solver->getNodeManager());
+ d_node.reset(new CVC4::Node(n));
}
Term::~Term()
generated by cgit on debian on lair
contact matthew@masot.net with questions or feedback