diff options
| author | Spencer Low <CompareAndSwap@gmail.com> | 2018-08-29 15:46:47 -0700 |
|---|---|---|
| committer | Spencer Low <CompareAndSwap@gmail.com> | 2018-08-29 15:46:47 -0700 |
| commit | c8ad0ca69d52cd53e30d1fb7dcbf93dd614db8ee (patch) | |
| tree | 05d24f5b4d550f0a7d423aa48b0d7c11864bafb9 /debuggerd/handler/debuggerd_handler.cpp | |
| parent | 6d2ace3684a06ea81d853657cbc6f5f0682667bc (diff) | |
adb: really make kill-server wait for server death
It doesn't seem like b3c14ec693d4c061fc9ec7fdaa7897169d179479 was really
waiting for adb server death because the server would write OKAY to the
socket, causing the client to return from ReadOrderlyShutdown() before
the socket was actually closed.
The fix is to read the OKAY and then call ReadOrderlyShutdown() which
will then wait for the socket to be closed.
Test: Made the server hang in adb_server_cleanup() during kill-server
and kill-server would hang (as desired).
Test: To test resilience, tried preventing the server from sending OKAY and
adb kill-server still worked properly.
Change-Id: I4f0f11b86bbdc983670273ef5d3bb1dff7ab3697
Signed-off-by: Spencer Low <CompareAndSwap@gmail.com>
Diffstat (limited to 'debuggerd/handler/debuggerd_handler.cpp')
0 files changed, 0 insertions, 0 deletions