diff options
| author | zork@chromium.org <zork@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-01-19 20:49:24 +0000 |
|---|---|---|
| committer | zork@chromium.org <zork@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-01-19 20:49:24 +0000 |
| commit | 35ee9505314a11dadac1bdfd41612f2984448af9 (patch) | |
| tree | dfa4e91b7203119e3cb6a66c2f1a06ff54510785 /chrome | |
| parent | daade9b87dd180e37a7dceaf2474776b9d7b8e5e (diff) | |
| download | chromium_src-35ee9505314a11dadac1bdfd41612f2984448af9.zip chromium_src-35ee9505314a11dadac1bdfd41612f2984448af9.tar.gz chromium_src-35ee9505314a11dadac1bdfd41612f2984448af9.tar.bz2 | |
Fix the linux perf build by checking the return value on write()
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/542116
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36540 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome')
| -rwxr-xr-x | chrome/browser/sync/engine/syncapi.cc | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/chrome/browser/sync/engine/syncapi.cc b/chrome/browser/sync/engine/syncapi.cc index 61e7db0..8dcf8c5 100755 --- a/chrome/browser/sync/engine/syncapi.cc +++ b/chrome/browser/sync/engine/syncapi.cc @@ -1231,7 +1231,12 @@ void SyncManager::SyncInternal::Shutdown() { SetEvent(address_watch_params_.exit_flag); #elif defined(OS_LINUX) char data = 0; - write(address_watch_params_.exit_pipe[1], &data, 1); + // We can't ignore the return value on write(), since that generates a compile + // warning. However, since we're exiting, there's nothing we can do if this + // fails except to log it. + if (write(address_watch_params_.exit_pipe[1], &data, 1) == -1) { + LOG(WARNING) << "Error sending error signal to AddressWatchTask"; + } close(address_watch_params_.exit_pipe[1]); #endif |