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

Side by Side Diff: sandbox/src/service_resolver.cc

Issue 3836005: Move pe_image and registry from base to base/win and use the namespace. It re... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 2 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/src/sandbox_nt_util.cc ('k') | sandbox/src/sidestep_resolver.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-2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2006-2010 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/src/service_resolver.h" 5 #include "sandbox/src/service_resolver.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "base/pe_image.h" 8 #include "base/win/pe_image.h"
9 9
10 namespace sandbox { 10 namespace sandbox {
11 11
12 NTSTATUS ServiceResolverThunk::ResolveInterceptor( 12 NTSTATUS ServiceResolverThunk::ResolveInterceptor(
13 const void* interceptor_module, 13 const void* interceptor_module,
14 const char* interceptor_name, 14 const char* interceptor_name,
15 const void** address) { 15 const void** address) {
16 // After all, we are using a locally mapped version of the exe, so the 16 // After all, we are using a locally mapped version of the exe, so the
17 // action is the same as for a target function. 17 // action is the same as for a target function.
18 return ResolveTarget(interceptor_module, interceptor_name, 18 return ResolveTarget(interceptor_module, interceptor_name,
19 const_cast<void**>(address)); 19 const_cast<void**>(address));
20 } 20 }
21 21
22 // In this case all the work is done from the parent, so resolve is 22 // In this case all the work is done from the parent, so resolve is
23 // just a simple GetProcAddress. 23 // just a simple GetProcAddress.
24 NTSTATUS ServiceResolverThunk::ResolveTarget(const void* module, 24 NTSTATUS ServiceResolverThunk::ResolveTarget(const void* module,
25 const char* function_name, 25 const char* function_name,
26 void** address) { 26 void** address) {
27 DCHECK(address); 27 DCHECK(address);
28 if (NULL == module) 28 if (NULL == module)
29 return STATUS_UNSUCCESSFUL; 29 return STATUS_UNSUCCESSFUL;
30 30
31 PEImage module_image(module); 31 base::win::PEImage module_image(module);
32 *address = module_image.GetProcAddress(function_name); 32 *address = module_image.GetProcAddress(function_name);
33 33
34 if (NULL == *address) { 34 if (NULL == *address) {
35 NOTREACHED(); 35 NOTREACHED();
36 return STATUS_UNSUCCESSFUL; 36 return STATUS_UNSUCCESSFUL;
37 } 37 }
38 38
39 return STATUS_SUCCESS; 39 return STATUS_SUCCESS;
40 } 40 }
41 41
42 } // namespace sandbox 42 } // namespace sandbox
OLDNEW
« no previous file with comments | « sandbox/src/sandbox_nt_util.cc ('k') | sandbox/src/sidestep_resolver.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698