diff options
| author | Josh Gao <jmgao@google.com> | 2016-05-18 10:39:48 -0700 |
|---|---|---|
| committer | gitbuildkicker <android-build@google.com> | 2016-08-16 15:52:46 -0700 |
| commit | 3b268646f802d72f9b53800ac00540201223cf2e (patch) | |
| tree | 5416950747cca0d710894309955499211a9f8378 | |
| parent | 11440395e379f35069e8e39a6e3ccfa7a7bfc72f (diff) | |
| download | system_core-3b268646f802d72f9b53800ac00540201223cf2e.zip system_core-3b268646f802d72f9b53800ac00540201223cf2e.tar.gz system_core-3b268646f802d72f9b53800ac00540201223cf2e.tar.bz2 | |
adb: use asocket's close function when closing.
close_all_sockets was assuming that all registered local sockets used
local_socket_close as their close function. However, this is not true
for JDWP sockets.
Bug: http://b/28347842
Change-Id: I40a1174845cd33f15f30ce70828a7081cd5a087e
(cherry picked from commit 53eb31d87cb84a4212f4850bf745646e1fb12814)
(cherry picked from commit 014b01706cc64dc9c2ad94a96f62e07c058d0b5d)
| -rw-r--r-- | adb/sockets.cpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/adb/sockets.cpp b/adb/sockets.cpp index 31b5443..3919147 100644 --- a/adb/sockets.cpp +++ b/adb/sockets.cpp @@ -43,8 +43,6 @@ using std::recursive_mutex; #endif -static void local_socket_close(asocket* s); - static recursive_mutex& local_socket_list_lock = *new recursive_mutex(); static unsigned local_socket_next_id = 1; @@ -128,7 +126,7 @@ void close_all_sockets(atransport *t) restart: for (s = local_socket_list.next; s != &local_socket_list; s = s->next) { if (s->transport == t || (s->peer && s->peer->transport == t)) { - local_socket_close(s); + s->close(s); goto restart; } } |