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

Unified Diff: sandbox/win/src/sandbox_policy.h

Issue 319573006: Add sandbox support for process memory limits (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: ready for review Created 6 years, 6 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
Index: sandbox/win/src/sandbox_policy.h
diff --git a/sandbox/win/src/sandbox_policy.h b/sandbox/win/src/sandbox_policy.h
index a9f124516218ae00cebb1f42cf58bd7250798234..3290630dd5c9aa78ac46f1874850f80bd9a6f4d3 100644
--- a/sandbox/win/src/sandbox_policy.h
+++ b/sandbox/win/src/sandbox_policy.h
@@ -122,6 +122,16 @@ class TargetPolicy {
// Note: the recommended level is JOB_RESTRICTED or JOB_LOCKDOWN.
virtual ResultCode SetJobLevel(JobLevel job_level, uint32 ui_exceptions) = 0;
+ // Sets a hard limit on the size of the commit set for the sandboxed process.
+ // If terminate is false, the allocation will fail when the limit is reached.
+ // If true, the process will terminate with SBOX_FATAL_MEMORY_EXCEEDED (7012)
+ // when the limit is reached.
cpu_(ooo_6.6-7.5) 2014/06/06 19:44:05 Not understanding this API. Say I have mapped a me
jschuh 2014/06/06 20:13:14 No, it's commit charge, which is the largest amoun
+ virtual ResultCode SetJobMemoryLimit(size_t limit, bool terminate) = 0;
+
+ // Returns whether the sandbox is set to terminate the process on hitting the
+ // memory limit imposed by the job object.
+ virtual bool WillTerminateOnJobMemoryLimit() const = 0;
+
// Specifies the desktop on which the application is going to run. If the
// desktop does not exist, it will be created. If alternate_winstation is
// set to true, the desktop will be created on an alternate window station.

Powered by Google App Engine
This is Rietveld 408576698