capstone
capstone copied to clipboard
Add test with ASAN enabled.
Your checklist for this pull request
- [ ] I've documented or updated the documentation of every function and struct this PR changes. If not so I've explained why.
- [x] I've added tests that prove my fix is effective or that my feature works (if possible)
Detailed description
Add ASAN builds
It does a few things
- Fix a lot of mem leaks,
- Add ASAN to the CI (this also removes the make build from the CI), because I did not want to add ASAN for a build type we consider deprecated. This can be debated and I can add it without ASAN support again.
- Fix the build of
cstestviaCMakeLists.txt. This allows now also to enable ASAN for the cstest. Also it is much easier to buildcstestnow. - Fixes cstest-report.py to work also when
cstestis installed and inPATH. - Make
run_invalid_cstool.shmore verbose to help spotting possible ASAN errors. - Add and fix
tests/cs_details/issue.csto the CI.
Test plan
CI green (or, if everything fails horribly, mark it optoinal for now)
Closing issues
...