diff options
| author | evan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-02-23 04:07:56 +0000 |
|---|---|---|
| committer | evan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-02-23 04:07:56 +0000 |
| commit | f4434b2ad164700510dc811f1e9bb30ea2c26bf9 (patch) | |
| tree | 1060e7e8b2dc2346e8705456a22a799dbdb9f831 /base/process_util_unittest.cc | |
| parent | 9e19aea882faeea78ce46911f201c5eac8e7c74d (diff) | |
| download | chromium_src-f4434b2ad164700510dc811f1e9bb30ea2c26bf9.zip chromium_src-f4434b2ad164700510dc811f1e9bb30ea2c26bf9.tar.gz chromium_src-f4434b2ad164700510dc811f1e9bb30ea2c26bf9.tar.bz2 | |
Fix more GCC 4.3 warnings.
Review URL: http://codereview.chromium.org/27038
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10170 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/process_util_unittest.cc')
| -rw-r--r-- | base/process_util_unittest.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/base/process_util_unittest.cc b/base/process_util_unittest.cc index 97ba756..c3bcea9 100644 --- a/base/process_util_unittest.cc +++ b/base/process_util_unittest.cc @@ -181,7 +181,8 @@ MULTIPROCESS_TEST_MAIN(ProcessUtilsLeakFDChildProcess) { #endif // defined(OS_LINUX) num_open_files -= expected_num_open_fds; - write(write_pipe, &num_open_files, sizeof(num_open_files)); + ssize_t written = write(write_pipe, &num_open_files, sizeof(num_open_files)); + DCHECK_EQ(written, sizeof(num_open_files)); close(write_pipe); return 0; @@ -190,7 +191,8 @@ MULTIPROCESS_TEST_MAIN(ProcessUtilsLeakFDChildProcess) { TEST_F(ProcessUtilTest, FDRemapping) { // Open some files to check they don't get leaked to the child process. int fds[2]; - pipe(fds); + if (pipe(fds) < 0) + NOTREACHED(); int pipe_read_fd = fds[0]; int pipe_write_fd = fds[1]; |