diff options
| author | rvargas@chromium.org <rvargas@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-12-20 10:18:01 +0000 |
|---|---|---|
| committer | rvargas@chromium.org <rvargas@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-12-20 10:18:01 +0000 |
| commit | db49c24130ce1523ea62f298564eb319f33a72b2 (patch) | |
| tree | 308394d155cc1968b14d280d384ec9409776577b /sandbox/win | |
| parent | 8168e65741433732460dd9851e28b2f8655c157b (diff) | |
| download | chromium_src-db49c24130ce1523ea62f298564eb319f33a72b2.zip chromium_src-db49c24130ce1523ea62f298564eb319f33a72b2.tar.gz chromium_src-db49c24130ce1523ea62f298564eb319f33a72b2.tar.bz2 | |
Sandbox: Don't create an ScopedHandle with an invalid handle.
BUG=166888
TEST=none
Review URL: https://chromiumcodereview.appspot.com/11639024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@174122 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'sandbox/win')
| -rw-r--r-- | sandbox/win/src/broker_services.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sandbox/win/src/broker_services.cc b/sandbox/win/src/broker_services.cc index 65a7b2b..f547a5b 100644 --- a/sandbox/win/src/broker_services.cc +++ b/sandbox/win/src/broker_services.cc @@ -309,10 +309,11 @@ ResultCode BrokerServicesBase::SpawnTarget(const wchar_t* exe_path, HANDLE job_temp; result = policy_base->MakeJobObject(&job_temp); - base::win::ScopedHandle job(job_temp); if (SBOX_ALL_OK != result) return result; + base::win::ScopedHandle job(job_temp); + // Initialize the startup information from the policy. base::win::StartupInformation startup_info; string16 desktop = policy_base->GetAlternateDesktop(); |