diff options
| author | rogerj <rogerj@opera.com> | 2015-06-12 11:17:44 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-06-12 18:18:21 +0000 |
| commit | b2134362e27459f1098a35effedad91e2d553192 (patch) | |
| tree | a67e7c691c95b114db9543c9bc04b8887c79afdd /media | |
| parent | f771be4a1fe6ddf853889d935402a60a5e394e99 (diff) | |
| download | chromium_src-b2134362e27459f1098a35effedad91e2d553192.zip chromium_src-b2134362e27459f1098a35effedad91e2d553192.tar.gz chromium_src-b2134362e27459f1098a35effedad91e2d553192.tar.bz2 | |
Fix leak of PlatformHandleDispatchers in Mojo IPC
PassWrappedPlatformHandle does not close the MojoHandle.
The destructor of ScopedHandleBase will however close it.
The problem with this code was that it called release()
on the ScopedHandle, preventing the destructor from being
called, and thus also the MojoHandle from being closed.
This leaves the PlatformHandleDispatcher associated with
the MojoHandle alive in mojo::system::Core::handle_table_
By calling reset() instead the MojoHandle is closed, and
the leak is fixed.
R=morrita@chromium.org, viettrungluu@chromium.org, agl@chromium.org
BUG=None
Review URL: https://codereview.chromium.org/1174423002
Cr-Commit-Position: refs/heads/master@{#334197}
Diffstat (limited to 'media')
0 files changed, 0 insertions, 0 deletions