339 stackoverflowerror using mathsat5

src/org/sosy_lab/java_smt/solvers/mathsat5/Mathsat5AbstractProver.java

@@ -24,6 +24,7 @@
 import static org.sosy_lab.java_smt.solvers.mathsat5.Mathsat5NativeApi.msat_pop_backtrack_point;
 import static org.sosy_lab.java_smt.solvers.mathsat5.Mathsat5NativeApi.msat_push_backtrack_point;
 import static org.sosy_lab.java_smt.solvers.mathsat5.Mathsat5NativeApi.msat_set_option_checked;
+import static org.sosy_lab.java_smt.solvers.mathsat5.Mathsat5NativeApi.msat_set_termination_callback;
 import static org.sosy_lab.java_smt.solvers.mathsat5.Mathsat5NativeApi.msat_term_get_arg;
 import static org.sosy_lab.java_smt.solvers.mathsat5.Mathsat5NativeApi.msat_term_is_boolean_constant;
 import static org.sosy_lab.java_smt.solvers.mathsat5.Mathsat5NativeApi.msat_term_is_not;
@@ -57,7 +58,6 @@ abstract class Mathsat5AbstractProver<T2> extends AbstractProver<T2> {
   protected final Mathsat5SolverContext context;
   protected final long curEnv;
   private final long curConfig;
-  private final long terminationTest;
   protected final Mathsat5FormulaCreator creator;
   private final ShutdownNotifier shutdownNotifier;
 
@@ -71,7 +71,6 @@ protected Mathsat5AbstractProver(
     creator = pCreator;
     curConfig = buildConfig(pOptions);
     curEnv = context.createEnvironment(curConfig);
-    terminationTest = context.addTerminationTest(curEnv);
     shutdownNotifier = pShutdownNotifier;
   }
 
@@ -100,15 +99,27 @@ private long buildConfig(Set<ProverOptions> opts) {
   @Override
   public boolean isUnsat() throws InterruptedException, SolverException {
     Preconditions.checkState(!closed);
-    return !msat_check_sat(curEnv);
+
+    final long hook = msat_set_termination_callback(curEnv, context.getTerminationTest());
+    try {
+      return !msat_check_sat(curEnv);
+    } finally {
+      msat_free_termination_callback(hook);
+    }
   }
 
   @Override
   public boolean isUnsatWithAssumptions(Collection<BooleanFormula> pAssumptions)
       throws SolverException, InterruptedException {
     Preconditions.checkState(!closed);
     checkForLiterals(pAssumptions);
-    return !msat_check_sat_with_assumptions(curEnv, getMsatTerm(pAssumptions));
+
+    final long hook = msat_set_termination_callback(curEnv, context.getTerminationTest());
+    try {
+      return !msat_check_sat_with_assumptions(curEnv, getMsatTerm(pAssumptions));
+    } finally {
+      msat_free_termination_callback(hook);
+    }
   }
 
   private void checkForLiterals(Collection<BooleanFormula> formulas) {
@@ -214,7 +225,6 @@ public ImmutableMap<String, String> getStatistics() {
   public void close() {
     if (!closed) {
       msat_destroy_env(curEnv);
-      msat_free_termination_callback(terminationTest);
       msat_destroy_config(curConfig);
     }
     super.close();

src/org/sosy_lab/java_smt/solvers/mathsat5/Mathsat5SolverContext.java

@@ -17,7 +17,6 @@
 import static org.sosy_lab.java_smt.solvers.mathsat5.Mathsat5NativeApi.msat_destroy_env;
 import static org.sosy_lab.java_smt.solvers.mathsat5.Mathsat5NativeApi.msat_get_version;
 import static org.sosy_lab.java_smt.solvers.mathsat5.Mathsat5NativeApi.msat_set_option_checked;
-import static org.sosy_lab.java_smt.solvers.mathsat5.Mathsat5NativeApi.msat_set_termination_callback;
 
 import com.google.common.annotations.VisibleForTesting;
 import com.google.common.base.Preconditions;
@@ -300,9 +299,13 @@ public void close() {
     }
   }
 
-  long addTerminationTest(long env) {
+  /**
+   * Get a termination callback for the current context. The callback can be registered upfront,
+   * i.e., before calling a possibly expensive computation in the solver to allow a proper shutdown.
+   */
+  TerminationCallback getTerminationTest() {
     Preconditions.checkState(!closed, "solver context is already closed");
-    return msat_set_termination_callback(env, terminationTest);
+    return terminationTest;
   }
 
   @Override