Comment #5 on issue 17027 by bugdro...@chromium.org: Crash -
tcmalloc::ThreadCache::FreeList::PopRange(int,void * *,void * *)
http://code.google.com/p/chromium/issues/detail?id=17027
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=23282
------------------------------------------------------------------------
r23282 | lafo...@chromium.org | 2009-08-12 22:12:04 -0700 (Wed, 12 Aug
2009) | 20 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/branches/195/src/base/pickle.cc?r1=23282&r2=23281
M
http://src.chromium.org/viewvc/chrome/branches/195/src/base/pickle.h?r1=23282&r2=23281
Merge 22826 - Add short circuit case for self assignment, and add virtual
destructor
Pickle is a base class of Message, which has a virtual destructor, so
it is better to have Pickle have a virtual destructor as well.
Add more defensive code in the case of self assignment.
**IF** self assignment was used, and the target was readonly,
then the existing code would have a memory fault.
I don't think this is currently possible in use cases, but
the defensize code is much better. (per suggestion from CPU
on last CL).
BUG=17027
BUG=17088
r=cpu
Review URL: http://codereview.chromium.org/159800
tbr=...@chromium.org
Review URL: http://codereview.chromium.org/165435
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---
