diff options
| author | piman@google.com <piman@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-02-15 00:52:07 +0000 |
|---|---|---|
| committer | piman@google.com <piman@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-02-15 00:52:07 +0000 |
| commit | 49c38da9a42f011de40b0a0f2464f6050e9ecf25 (patch) | |
| tree | 62845e8a5f83346a999da49584eb620d1703f466 /ppapi | |
| parent | 2751df0d75c66c1f87e899e0c746a39462f552ce (diff) | |
| download | chromium_src-49c38da9a42f011de40b0a0f2464f6050e9ecf25.zip chromium_src-49c38da9a42f011de40b0a0f2464f6050e9ecf25.tar.gz chromium_src-49c38da9a42f011de40b0a0f2464f6050e9ecf25.tar.bz2 | |
Fix PPB_Audio_Proxy handling of file descriptors.
Previous code was closing a file descriptor it shouldn't causing trouble when the real owner tried to close it again.
BUG=none
TEST=with pepper flash, load a youtube video with sound, reload the page, should see no assert.
Review URL: http://codereview.chromium.org/6528004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74894 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ppapi')
| -rw-r--r-- | ppapi/proxy/ppb_audio_proxy.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ppapi/proxy/ppb_audio_proxy.cc b/ppapi/proxy/ppb_audio_proxy.cc index 6adc436..3e7350b 100644 --- a/ppapi/proxy/ppb_audio_proxy.cc +++ b/ppapi/proxy/ppb_audio_proxy.cc @@ -282,8 +282,8 @@ int32_t PPB_Audio_Proxy::GetAudioConnectedHandles( FALSE, DUPLICATE_CLOSE_SOURCE); #else // On Posix, the socket handle will be auto-duplicated when we send the - // FileDescriptor. Set AutoClose since we don't need the handle any more. - *foreign_socket_handle = base::FileDescriptor(socket_handle, true); + // FileDescriptor. Don't set AutoClose since this is not our handle. + *foreign_socket_handle = base::FileDescriptor(socket_handle, false); #endif // Get the shared memory for the buffer. |