diff options
| author | stoyan@chromium.org <stoyan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-02-09 19:28:41 +0000 |
|---|---|---|
| committer | stoyan@chromium.org <stoyan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-02-09 19:28:41 +0000 |
| commit | cd4fd156aba80ddffd3fffd8e2d8703247181fe0 (patch) | |
| tree | 1f4b2dc5f58e5f534b033f71e0e48cc5549c6732 /chrome/common/ipc_fuzzing_tests.cc | |
| parent | a6309f8198825442d77f0fc9aa70c82ae5dd6d04 (diff) | |
| download | chromium_src-cd4fd156aba80ddffd3fffd8e2d8703247181fe0.zip chromium_src-cd4fd156aba80ddffd3fffd8e2d8703247181fe0.tar.gz chromium_src-cd4fd156aba80ddffd3fffd8e2d8703247181fe0.tar.bz2 | |
Fix the windows implementation of KillProcess and WaitForSingleProcess to not close the process handle that they do not own.
Review URL: http://codereview.chromium.org/24004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@9400 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/common/ipc_fuzzing_tests.cc')
| -rw-r--r-- | chrome/common/ipc_fuzzing_tests.cc | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/chrome/common/ipc_fuzzing_tests.cc b/chrome/common/ipc_fuzzing_tests.cc index 0638bb4..b9a9563 100644 --- a/chrome/common/ipc_fuzzing_tests.cc +++ b/chrome/common/ipc_fuzzing_tests.cc @@ -302,6 +302,7 @@ TEST_F(IPCFuzzingTest, SanityTest) { EXPECT_TRUE(listener.ExpectMessage(value, MsgClassSI::ID)); EXPECT_TRUE(base::WaitForSingleProcess(server_process, 5000)); + base::CloseProcessHandle(server_process); } // This test uses a payload that is smaller than expected. @@ -331,6 +332,7 @@ TEST_F(IPCFuzzingTest, MsgBadPayloadShort) { EXPECT_TRUE(listener.ExpectMessage(1, MsgClassSI::ID)); EXPECT_TRUE(base::WaitForSingleProcess(server_process, 5000)); + base::CloseProcessHandle(server_process); } #endif // NDEBUG @@ -365,6 +367,7 @@ TEST_F(IPCFuzzingTest, MsgBadPayloadArgs) { EXPECT_TRUE(listener.ExpectMessage(3, MsgClassIS::ID)); EXPECT_TRUE(base::WaitForSingleProcess(server_process, 5000)); + base::CloseProcessHandle(server_process); } // This class is for testing the IPC_BEGIN_MESSAGE_MAP_EX macros. |