Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: segfault in minisat #75

Merged
merged 4 commits into from
Apr 4, 2024
Merged

fix: segfault in minisat #75

merged 4 commits into from
Apr 4, 2024

Conversation

chrjabs
Copy link
Owner

@chrjabs chrjabs commented Mar 29, 2024

newVar method was not virtual so core::newVar was called instead of simp::newVar

resolves #74

`newVar` method was not virtual so `core::newVar` was called instead of
`simp::newVar`

resolves #74
@chrjabs
Copy link
Owner Author

chrjabs commented Mar 29, 2024

Add tests that would have caught this before merging.

@chrjabs chrjabs force-pushed the bugfix/minisat-simp-segfault branch 2 times, most recently from a72fe66 to ddd6e61 Compare April 4, 2024 08:16
@chrjabs
Copy link
Owner Author

chrjabs commented Apr 4, 2024

Also resolves #76

@chrjabs chrjabs force-pushed the bugfix/minisat-simp-segfault branch from b71aba0 to f78c6d2 Compare April 4, 2024 08:25
@chrjabs chrjabs merged commit d768c48 into main Apr 4, 2024
36 checks passed
@chrjabs chrjabs deleted the bugfix/minisat-simp-segfault branch April 4, 2024 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

rustsat_minisat::simp segmentation fault
1 participant