diff options
| author | jschuh@chromium.org <jschuh@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-08-15 01:05:28 +0000 |
|---|---|---|
| committer | jschuh@chromium.org <jschuh@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-08-15 01:05:28 +0000 |
| commit | cf7cb81e6c68f0cb80d34f8818c5aecc7495e7de (patch) | |
| tree | 170c3ea387df7de92b1894f01c6d4cb74e6b7deb /sandbox | |
| parent | 02117ea8cc1973873c72ef2a836135586a67c76e (diff) | |
| download | chromium_src-cf7cb81e6c68f0cb80d34f8818c5aecc7495e7de.zip chromium_src-cf7cb81e6c68f0cb80d34f8818c5aecc7495e7de.tar.gz chromium_src-cf7cb81e6c68f0cb80d34f8818c5aecc7495e7de.tar.bz2 | |
Fix PVS assignment warning in DuplicateHandleProxyAction
The existing assignment is safe because there's no implicit
conversion from HANDLE to ScopedHandle, whereas there is a
HANDLE cast operator for ScopedHandle. However, this change
makes the conversion clearer and silences the PVS warning.
BUG=271530
R=cpu@chromium.org
Review URL: https://chromiumcodereview.appspot.com/23043005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@217719 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'sandbox')
| -rw-r--r-- | sandbox/win/src/handle_policy.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sandbox/win/src/handle_policy.cc b/sandbox/win/src/handle_policy.cc index eeeea7b..718376e 100644 --- a/sandbox/win/src/handle_policy.cc +++ b/sandbox/win/src/handle_policy.cc @@ -79,7 +79,7 @@ DWORD HandlePolicy::DuplicateHandleProxyAction(EvalResult eval_result, // If the policy didn't block us and we have no valid target, then the broker // (this process) is the valid target. HANDLE target_process = remote_target_process.IsValid() ? - remote_target_process : ::GetCurrentProcess(); + remote_target_process.Get() : ::GetCurrentProcess(); DWORD result = ERROR_SUCCESS; if (!::DuplicateHandle(client_info.process, source_handle, target_process, target_handle, desired_access, FALSE, |