Index: sandbox/win/src/process_mitigations_test.cc |
diff --git a/sandbox/win/src/process_mitigations_test.cc b/sandbox/win/src/process_mitigations_test.cc |
index 4d2e9c6e9562a6abdd3488afaa2a4636cc893fac..4b3d3956973d08b9888f5a030f77aa49ec5610f8 100644 |
--- a/sandbox/win/src/process_mitigations_test.cc |
+++ b/sandbox/win/src/process_mitigations_test.cc |
@@ -165,9 +165,9 @@ SBOX_TESTS_COMMAND int CheckDep(int argc, wchar_t **argv) { |
return SBOX_TEST_THIRD_ERROR; |
} |
- static const int MEM_EXECUTE_OPTION_ENABLE = 1; |
+ // static const int MEM_EXECUTE_OPTION_ENABLE = 1; |
Nico
2015/07/02 17:34:26
?
Matt Giuca
2015/07/03 03:46:29
Well I thought that since this seems to represent
Nico
2015/07/03 04:11:15
As-is, it looks like code that was left in acciden
Matt Giuca
2015/07/03 04:33:45
jschuh: Can I get your quick opinion on this. You
|
static const int MEM_EXECUTE_OPTION_DISABLE = 2; |
- static const int MEM_EXECUTE_OPTION_ATL7_THUNK_EMULATION = 4; |
+ // static const int MEM_EXECUTE_OPTION_ATL7_THUNK_EMULATION = 4; |
Nico
2015/07/02 17:34:26
?
jschuh
2015/07/06 22:16:18
Yeah, looks like dead constants. lgtm on removing
Matt Giuca
2015/07/07 04:52:17
Well they were dead the moment they were added, so
|
static const int MEM_EXECUTE_OPTION_PERMANENT = 8; |
dep_flags &= 0xff; |