Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(184)

Unified Diff: third_party/tcmalloc/chromium/src/free_list.cc

Issue 9528002: Try adding memory corruption check in ThreadCache::Deallocate. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Fix the memory corruption check. Created 8 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | third_party/tcmalloc/chromium/src/thread_cache.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/tcmalloc/chromium/src/free_list.cc
diff --git a/third_party/tcmalloc/chromium/src/free_list.cc b/third_party/tcmalloc/chromium/src/free_list.cc
index 842f391c4593071dee2127063fde5a05f78f6913..61e9a1d1a36969b0978d1a01861b5b2bb6af8431 100644
--- a/third_party/tcmalloc/chromium/src/free_list.cc
+++ b/third_party/tcmalloc/chromium/src/free_list.cc
@@ -150,6 +150,7 @@ void *FL_Pop(void **list) {
if (*list != NULL) {
FL_SetPrevious(*list, NULL);
}
+ // Maybe required here? : FL_SetNext(result, NULL);
jar (doing other things) 2012/03/01 19:36:38 An item pulled from the list is not guaranteed to
Dai Mikurube (NOT FULLTIME) 2012/03/01 21:04:11 Make sense. I don't change here.
return result;
}
« no previous file with comments | « no previous file | third_party/tcmalloc/chromium/src/thread_cache.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698