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

Side by Side Diff: base/process/memory.h

Issue 217343002: Update code related to OOM errors in sanitizer builds. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 8 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | base/process/memory_linux.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef BASE_PROCESS_MEMORY_H_ 5 #ifndef BASE_PROCESS_MEMORY_H_
6 #define BASE_PROCESS_MEMORY_H_ 6 #define BASE_PROCESS_MEMORY_H_
7 7
8 #include "base/base_export.h" 8 #include "base/base_export.h"
9 #include "base/basictypes.h" 9 #include "base/basictypes.h"
10 #include "base/process/process_handle.h" 10 #include "base/process/process_handle.h"
(...skipping 49 matching lines...) Expand 10 before | Expand all | Expand 10 after
60 // may occur in that case, of course. 60 // may occur in that case, of course.
61 BASE_EXPORT bool AdjustOOMScore(ProcessId process, int score); 61 BASE_EXPORT bool AdjustOOMScore(ProcessId process, int score);
62 #endif 62 #endif
63 63
64 // Special allocator functions for callers that want to check for OOM. 64 // Special allocator functions for callers that want to check for OOM.
65 // These will not abort if the allocation fails even if 65 // These will not abort if the allocation fails even if
66 // EnableTerminationOnOutOfMemory has been called. 66 // EnableTerminationOnOutOfMemory has been called.
67 // This can be useful for huge and/or unpredictable size memory allocations. 67 // This can be useful for huge and/or unpredictable size memory allocations.
68 // Please only use this if you really handle the case when the allocation 68 // Please only use this if you really handle the case when the allocation
69 // fails. Doing otherwise would risk security. 69 // fails. Doing otherwise would risk security.
70 // These functions may still crash on OOM when running under memory tools,
71 // specifically ASan and other sanitizers.
70 // Return value tells whether the allocation succeeded. If it fails |result| is 72 // Return value tells whether the allocation succeeded. If it fails |result| is
71 // set to NULL, otherwise it holds the memory address. 73 // set to NULL, otherwise it holds the memory address.
72 BASE_EXPORT WARN_UNUSED_RESULT bool UncheckedMalloc(size_t size, 74 BASE_EXPORT WARN_UNUSED_RESULT bool UncheckedMalloc(size_t size,
73 void** result); 75 void** result);
74 BASE_EXPORT WARN_UNUSED_RESULT bool UncheckedCalloc(size_t num_items, 76 BASE_EXPORT WARN_UNUSED_RESULT bool UncheckedCalloc(size_t num_items,
75 size_t size, 77 size_t size,
76 void** result); 78 void** result);
77 79
78 // TODO(b.kelemen): make Skia use the new interface and remove these. 80 // TODO(b.kelemen): make Skia use the new interface and remove these.
79 #if defined(OS_MACOSX) 81 #if defined(OS_MACOSX)
80 BASE_EXPORT void* UncheckedMalloc(size_t size); 82 BASE_EXPORT void* UncheckedMalloc(size_t size);
81 BASE_EXPORT void* UncheckedCalloc(size_t num_items, size_t size); 83 BASE_EXPORT void* UncheckedCalloc(size_t num_items, size_t size);
82 #endif 84 #endif
83 85
84 } // namespace base 86 } // namespace base
85 87
86 #endif // BASE_PROCESS_MEMORY_H_ 88 #endif // BASE_PROCESS_MEMORY_H_
OLDNEW
« no previous file with comments | « no previous file | base/process/memory_linux.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698