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 crash because of mixing debug and release CRT. #1361

Merged

Conversation

Projects
None yet
2 participants
@a1ext
Copy link
Member

a1ext commented Mar 22, 2019

Memory allocated in r2-side has to be released with r_free

Test plan (required)

  1. Build r2 with help of prepare_r2.bat (will have Release configuration by-default)
  2. build Cutter in debug mode
  3. Run it, it hasn't crash
Fix crash because of mixing debug and release CRT.
Memory allocated in r2-side has to be released with `r_free`
@ITAYC0HEN
Copy link
Member

ITAYC0HEN left a comment

Thank you very much :)

@ITAYC0HEN ITAYC0HEN merged commit 1422013 into radareorg:master Mar 23, 2019

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.