Message155259
So the risk is that descr may be given garbage memory that just happens to look like an object with 1 reference, so that it really does a decref and tries to re-deallocate whatever was there before.
I would rather see the change at the top of the function; just initialize descr to NULL when it is declared, and let the compiler figure out that it can skip the zeroing on some (most) paths. |
|
Date |
User |
Action |
Args |
2012-03-09 19:23:14 | Jim.Jewett | set | recipients:
+ Jim.Jewett, benjamin.peterson, Mark.Shannon |
2012-03-09 19:23:14 | Jim.Jewett | set | messageid: <1331320994.12.0.629991211548.issue14239@psf.upfronthosting.co.za> |
2012-03-09 19:23:13 | Jim.Jewett | link | issue14239 messages |
2012-03-09 19:23:13 | Jim.Jewett | create | |
|