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.

Sign up for GitHub

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

Jump to bottom

Use of free instead of delete in bitcoinrpc.cpp #822

Closed
xanatos opened this issue Feb 11, 2012 · 0 comments
Closed

Use of free instead of delete in bitcoinrpc.cpp #822

xanatos opened this issue Feb 11, 2012 · 0 comments

Comments

@xanatos
Copy link

xanatos commented Feb 11, 2012

bitcoinrpc.cpp, line 1564. parg is freed through free (free(parg);), but it was allocated with new (line 1475 int* pnSleepTime = new int(params[1].get_int());). It is correct in line 1440 (delete (int*)parg;)
@sipa sipa closed this as completed in 402deef Feb 11, 2012
sipa pushed a commit to sipa/bitcoin that referenced this issue Feb 11, 2012
@gavinandresen
Merge pull request bitcoin#825 from sipa/fix_822
b25474d 
Fix bitcoin#822: use free instead of delete
coblee referenced this issue in litecoin-project/litecoin Jul 17, 2012
@sipa
Fix #822
cc73517
coblee referenced this issue in litecoin-project/litecoin Jul 17, 2012
@gavinandresen
Merge pull request #825 from sipa/fix_822
3cd9445 
Fix #822: use free instead of delete
destenson pushed a commit to destenson/bitcoin--bitcoin that referenced this issue Jun 26, 2016
@crowning- @schinzelh
Merge bitcoin#822: [Core]: fix restart hanging during wallet-repair
ee830ee 
37a934c [Core]: fix restart hanging during wallet-repair
lateminer pushed a commit to lateminer/bitcoin that referenced this issue Apr 6, 2019
@Fuzzbawls
Merge bitcoin#822: [Tests] Integrate fake stake tests into parent tes…
2b8daac 
…t suite

52b5094 Remove stale qtum python scripts (Fuzzbawls)
c0d8dcb [Tests] Integrate fake stake tests into parent test suite (Fuzzbawls)

Tree-SHA512: b0f9d559b3a8a4d0da3d400dfbe57fa15933a3c569732c4d7d10e147e7d5cd5f70a6f5231c862e4ca5f7f3efdb2de5890fe4755cbee5c41b00a4ab34a015c2e4
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sipa @xanatos