diff options
| author | Josh Gao <jmgao@google.com> | 2017-09-13 13:40:57 -0700 |
|---|---|---|
| committer | Josh Gao <jmgao@google.com> | 2017-09-13 14:31:14 -0700 |
| commit | e48ecce671a2ab6647cc26a3cb11163aec0b712c (patch) | |
| tree | dd5648c4e6afcce2f8391f1e60be36e43489d122 /libsparse/sparse_read.cpp | |
| parent | 62c92f0c0529d8d1817afb4ff0c83151e4bb0ea0 (diff) | |
Revert "adb: fix deadlock between transport_unref and usb_close."
This reverts commit 7e197ef83307ffcf2f55d1b8a603d0d3bd2e1d4d.
The mutex lock in transport_unref hides a race that seems otherwise
hard to fix. Specifically, there's no synchronization between acquiring
a transport and attaching it to an asocket*, leading to badness if the
transport is closed in between the two operations.
Fix the original problem the reverted patch addressed by manually
unlocking before calling unregister_usb_transport.
Bug: http://b/65419665
Test: python test_device.py
Change-Id: I0ed0044129b1671b2c5dd1b9fa2e70a9b4475dc5
Diffstat (limited to 'libsparse/sparse_read.cpp')
0 files changed, 0 insertions, 0 deletions