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

Side by Side Diff: src/trusted/nonnacl_util/sel_ldr_launcher_standalone.cc

Issue 10832040: Support 64-bit reserved_at_zero value nacl-side (Closed) Base URL: http://git.chromium.org/native_client/src/native_client.git@master
Patch Set: Created 8 years, 4 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
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2012 The Native Client Authors. All rights reserved. 2 * Copyright (c) 2012 The Native Client Authors. All rights reserved.
3 * Use of this source code is governed by a BSD-style license that can be 3 * Use of this source code is governed by a BSD-style license that can be
4 * found in the LICENSE file. 4 * found in the LICENSE file.
5 */ 5 */
6 6
7 #include <assert.h> 7 #include <assert.h>
8 8
9 #include <algorithm> 9 #include <algorithm>
10 #include <iterator> 10 #include <iterator>
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
52 52
53 void SelLdrLauncherStandalone::BuildCommandLine(vector<nacl::string>* command) { 53 void SelLdrLauncherStandalone::BuildCommandLine(vector<nacl::string>* command) {
54 assert(sel_ldr_ != NACL_NO_FILE_PATH); // Set by InitCommandLine(). 54 assert(sel_ldr_ != NACL_NO_FILE_PATH); // Set by InitCommandLine().
55 if (!command_prefix_.empty()) 55 if (!command_prefix_.empty())
56 command->insert(command->end(), 56 command->insert(command->end(),
57 command_prefix_.begin(), command_prefix_.end()); 57 command_prefix_.begin(), command_prefix_.end());
58 if (!sel_ldr_bootstrap_.empty()) 58 if (!sel_ldr_bootstrap_.empty())
59 command->push_back(sel_ldr_bootstrap_); 59 command->push_back(sel_ldr_bootstrap_);
60 command->push_back(sel_ldr_); 60 command->push_back(sel_ldr_);
61 if (!sel_ldr_bootstrap_.empty()) { 61 if (!sel_ldr_bootstrap_.empty()) {
62 command->push_back("--r_debug=0xXXXXXXXXXXXXXXXX"); 62 static const nacl::string kTemplateDigits = "XXXXXXXXXXXXXXXX";
63 command->push_back("--reserved_at_zero=0xXXXXXXXX"); 63 command->push_back("--r_debug=0x" + kTemplateDigits);
Mark Seaborn 2012/07/30 18:51:12 You could avoid a runtime concatenation by inlinin
Austin Benson 2012/07/31 23:24:12 Inlined.
64 command->push_back("--reserved_at_zero=0x" + kTemplateDigits);
64 } 65 }
65 command->push_back("-R"); // RPC will be used to point to the nexe. 66 command->push_back("-R"); // RPC will be used to point to the nexe.
66 67
67 command->insert(command->end(), sel_ldr_argv_.begin(), sel_ldr_argv_.end()); 68 command->insert(command->end(), sel_ldr_argv_.begin(), sel_ldr_argv_.end());
68 69
69 struct NaClEnvCleanser env_cleanser; 70 struct NaClEnvCleanser env_cleanser;
70 NaClEnvCleanserCtor(&env_cleanser, 1); 71 NaClEnvCleanserCtor(&env_cleanser, 1);
71 if (!NaClEnvCleanserInit(&env_cleanser, GetEnviron(), NULL)) { 72 if (!NaClEnvCleanserInit(&env_cleanser, GetEnviron(), NULL)) {
72 NaClLog(LOG_FATAL, "Failed to initialise env cleanser\n"); 73 NaClLog(LOG_FATAL, "Failed to initialise env cleanser\n");
73 } 74 }
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
142 args_for_sel_ldr.push_back("-B"); 143 args_for_sel_ldr.push_back("-B");
143 args_for_sel_ldr.push_back(irt_library_path); 144 args_for_sel_ldr.push_back(irt_library_path);
144 } 145 }
145 if (!StartViaCommandLine(prefix, args_for_sel_ldr, args_for_nexe)) { 146 if (!StartViaCommandLine(prefix, args_for_sel_ldr, args_for_nexe)) {
146 return false; 147 return false;
147 } 148 }
148 return true; 149 return true;
149 } 150 }
150 151
151 } // namespace nacl 152 } // namespace nacl
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698