| Index: components/nacl/broker/nacl_broker_listener.cc
|
| diff --git a/components/nacl/broker/nacl_broker_listener.cc b/components/nacl/broker/nacl_broker_listener.cc
|
| index 2704695a4bc3a1e038e969943f008a920962b59c..e7fa3d1a530775653b1c7e91c980245da3dbddb7 100644
|
| --- a/components/nacl/broker/nacl_broker_listener.cc
|
| +++ b/components/nacl/broker/nacl_broker_listener.cc
|
| @@ -109,9 +109,11 @@ void NaClBrokerListener::OnLaunchLoaderThroughBroker(
|
| cmd_line->AppendSwitchASCII(switches::kProcessChannelID,
|
| loader_channel_id);
|
|
|
| - base::Process loader_process = content::StartSandboxedProcess(
|
| - this, cmd_line, base::HandlesToInheritVector());
|
| - if (loader_process.IsValid()) {
|
| + base::Process loader_process;
|
| + sandbox::ResultCode result = content::StartSandboxedProcess(
|
| + this, cmd_line, base::HandlesToInheritVector(), &loader_process);
|
| +
|
| + if (result == sandbox::SBOX_ALL_OK) {
|
| // Note: PROCESS_DUP_HANDLE is necessary here, because:
|
| // 1) The current process is the broker, which is the loader's parent.
|
| // 2) The browser is not the loader's parent, and so only gets the
|
|
|