diff options
author | dcheng <dcheng@chromium.org> | 2015-10-09 18:17:26 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-10-10 01:18:27 +0000 |
commit | 7764fe9ea52233096b7916614eb7c60b26bbca05 (patch) | |
tree | 0f7d3570c76e7387f1f329045c55fe70cce69971 /sandbox/win | |
parent | fbe0354f403ff793ccacd4d1e162583c11bd58db (diff) | |
download | chromium_src-7764fe9ea52233096b7916614eb7c60b26bbca05.zip chromium_src-7764fe9ea52233096b7916614eb7c60b26bbca05.tar.gz chromium_src-7764fe9ea52233096b7916614eb7c60b26bbca05.tar.bz2 |
Enforce WARN_UNUSED_RESULT attribute on Windows
BUG=none
R=cpu,danakj,dpranke,hans,kelvinp,rickyz,sky,sorin,thestig,tuttle,yukawa
TBR=vitalybuka
Review URL: https://codereview.chromium.org/1390223002
Cr-Commit-Position: refs/heads/master@{#353450}
Diffstat (limited to 'sandbox/win')
-rw-r--r-- | sandbox/win/src/broker_services.cc | 3 | ||||
-rw-r--r-- | sandbox/win/src/target_process.cc | 3 |
2 files changed, 4 insertions, 2 deletions
diff --git a/sandbox/win/src/broker_services.cc b/sandbox/win/src/broker_services.cc index 92cd0e0..9579daa 100644 --- a/sandbox/win/src/broker_services.cc +++ b/sandbox/win/src/broker_services.cc @@ -7,6 +7,7 @@ #include <AclAPI.h> #include "base/logging.h" +#include "base/macros.h" #include "base/memory/scoped_ptr.h" #include "base/stl_util.h" #include "base/threading/platform_thread.h" @@ -551,7 +552,7 @@ ResultCode BrokerServicesBase::AddTargetPeer(HANDLE peer_process) { } // Release the pointer since it will be cleaned up by the callback. - peer.release(); + ignore_result(peer.release()); return SBOX_ALL_OK; } diff --git a/sandbox/win/src/target_process.cc b/sandbox/win/src/target_process.cc index 84d76b0..416713f 100644 --- a/sandbox/win/src/target_process.cc +++ b/sandbox/win/src/target_process.cc @@ -5,6 +5,7 @@ #include "sandbox/win/src/target_process.h" #include "base/basictypes.h" +#include "base/macros.h" #include "base/memory/scoped_ptr.h" #include "base/win/pe_image.h" #include "base/win/startup_information.h" @@ -99,7 +100,7 @@ TargetProcess::~TargetProcess() { // that. if (shared_section_.IsValid()) shared_section_.Take(); - ipc_server_.release(); + ignore_result(ipc_server_.release()); sandbox_process_info_.TakeProcessHandle(); return; } |