Fix #822: use free instead of delete #825

pull sipa wants to merge 1 commits into bitcoin:master from sipa:fix_822 changing 1 files +1 −1

sipa commented at 2:27 PM on February 11, 2012: member

No description provided.
Fix #822 402deef944
laanwj commented at 2:31 PM on February 11, 2012: member

ACK. This might solve a few random RPC crashes.
sipa commented at 4:09 PM on February 11, 2012: member

I believe delete of int* is identical to just free, so i doubt it will change anything in practice.
gavinandresen referenced this in commit b25474d1be on Feb 11, 2012
gavinandresen merged this on Feb 11, 2012
gavinandresen closed this on Feb 11, 2012
laanwj commented at 5:20 PM on February 11, 2012: member

@sipa it's not -- it might work with some compilers, but not by definition in the C++ standard, doing so gives "undefined behavior"
sipa commented at 7:56 PM on February 11, 2012: member

Sure... it certainly requires fixing because of that, but I doubt it's a problem in practice for the platforms we're using.
coblee referenced this in commit 3cd9445298 on Jul 17, 2012
destenson referenced this in commit 989317c036 on Jun 26, 2016
lateminer referenced this in commit 0f1764a3db on Oct 30, 2019
Losangelosgenetics referenced this in commit aa4a8cda23 on Mar 12, 2020
DrahtBot locked this on Sep 8, 2021

Contributors