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

Side by Side Diff: sandbox/win/src/policy_engine_unittest.cc

Issue 294753003: Fix clang -Wwritable-strings warnings in the Windows sandbox (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 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 | « sandbox/win/src/named_pipe_policy_test.cc ('k') | sandbox/win/src/policy_low_level_unittest.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) 2006-2008 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2006-2008 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 #include "sandbox/win/src/policy_engine_params.h" 5 #include "sandbox/win/src/policy_engine_params.h"
6 #include "sandbox/win/src/policy_engine_processor.h" 6 #include "sandbox/win/src/policy_engine_processor.h"
7 #include "testing/gtest/include/gtest/gtest.h" 7 #include "testing/gtest/include/gtest/gtest.h"
8 8
9 #define POLPARAMS_BEGIN(x) sandbox::ParameterSet x[] = { 9 #define POLPARAMS_BEGIN(x) sandbox::ParameterSet x[] = {
10 #define POLPARAM(p) sandbox::ParamPickerMake(p), 10 #define POLPARAM(p) sandbox::ParamPickerMake(p),
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
53 opcode_maker.MakeOpAction(ASK_BROKER, kPolNone); 53 opcode_maker.MakeOpAction(ASK_BROKER, kPolNone);
54 54
55 // Add rule set #2 55 // Add rule set #2
56 opcode_maker.MakeOpWStringMatch(FileNameArg, L".TXT", 56 opcode_maker.MakeOpWStringMatch(FileNameArg, L".TXT",
57 kSeekToEnd, CASE_INSENSITIVE, kPolNone); 57 kSeekToEnd, CASE_INSENSITIVE, kPolNone);
58 opcode_maker.MakeOpNumberMatch(CreationDispositionArg, CREATE_NEW, 58 opcode_maker.MakeOpNumberMatch(CreationDispositionArg, CREATE_NEW,
59 kPolNegateEval); 59 kPolNegateEval);
60 opcode_maker.MakeOpAction(FAKE_ACCESS_DENIED, kPolNone); 60 opcode_maker.MakeOpAction(FAKE_ACCESS_DENIED, kPolNone);
61 policy->opcode_count = 7; 61 policy->opcode_count = 7;
62 62
63 wchar_t* filename = L"c:\\Documents and Settings\\Microsoft\\BLAH.txt"; 63 const wchar_t* filename = L"c:\\Documents and Settings\\Microsoft\\BLAH.txt";
64 unsigned long creation_mode = OPEN_EXISTING; 64 unsigned long creation_mode = OPEN_EXISTING;
65 unsigned long flags = FILE_ATTRIBUTE_NORMAL; 65 unsigned long flags = FILE_ATTRIBUTE_NORMAL;
66 void* security_descriptor = NULL; 66 void* security_descriptor = NULL;
67 67
68 POLPARAMS_BEGIN(eval_params) 68 POLPARAMS_BEGIN(eval_params)
69 POLPARAM(filename) 69 POLPARAM(filename)
70 POLPARAM(creation_mode) 70 POLPARAM(creation_mode)
71 POLPARAM(flags) 71 POLPARAM(flags)
72 POLPARAM(security_descriptor) 72 POLPARAM(security_descriptor)
73 POLPARAMS_END; 73 POLPARAMS_END;
(...skipping 19 matching lines...) Expand all
93 // Changing creation_mode such that evaluation should match rule #2. 93 // Changing creation_mode such that evaluation should match rule #2.
94 creation_mode = OPEN_ALWAYS; 94 creation_mode = OPEN_ALWAYS;
95 pr = pol_ev.Evaluate(kShortEval, eval_params, _countof(eval_params)); 95 pr = pol_ev.Evaluate(kShortEval, eval_params, _countof(eval_params));
96 EXPECT_EQ(POLICY_MATCH, pr); 96 EXPECT_EQ(POLICY_MATCH, pr);
97 EXPECT_EQ(FAKE_ACCESS_DENIED, pol_ev.GetAction()); 97 EXPECT_EQ(FAKE_ACCESS_DENIED, pol_ev.GetAction());
98 98
99 delete [] reinterpret_cast<char*>(policy); 99 delete [] reinterpret_cast<char*>(policy);
100 } 100 }
101 101
102 } // namespace sandbox 102 } // namespace sandbox
OLDNEW
« no previous file with comments | « sandbox/win/src/named_pipe_policy_test.cc ('k') | sandbox/win/src/policy_low_level_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698