Commit 12d7f090 authored Aug 28, 2015 by Yabin Cui

adb: call run_transport_disconnects() only once.

Currently run_transport_disconnects() are called twice. One is in
handle_offline(), another is before destroying transport.
The users of disconnect callback are listener, adb_auth_client, and
remote_sockets. All of them need only to be called once. And after
handle_offline, no new listeners, adb_auth_client, or remote_sockets
can be connected to the offlined transport. So I think we can remove
the second call to run_transport_disconnects().

Change-Id: I1ef8b6b7b5ab7ae1bad109be107c85973d65a2e3

parent 2c58af52

adb/transport.cpp

+0 −2

Original line number	Diff line number	Diff line
		@@ -536,8 +536,6 @@ static void transport_registration_func(int _fd, unsigned ev, void *data)
		transport_list.remove(t);
		adb_mutex_unlock(&transport_lock);

		run_transport_disconnects(t);

		if (t->product)
		free(t->product);
		if (t->serial)