diff options
author | Josh Gao <jmgao@google.com> | 2016-02-23 18:05:57 -0800 |
---|---|---|
committer | Josh Gao <jmgao@google.com> | 2016-02-24 17:56:12 -0800 |
commit | 196ed8c14f801605a5033527b45eaab995d8f41e (patch) | |
tree | 27916ceb8c4aaa16098115601e570c31f4f56af3 /transport.h | |
parent | ab63a5078d5212f913c26125138a735be3050d01 (diff) | |
download | adb-196ed8c14f801605a5033527b45eaab995d8f41e.tar.gz |
adb: fix leak of framework_fd.
Move the fdevent for the framework authentication connection out of
atransport into its own static variable in adb_auth_client, since its
lifetime is completely unrelated to that of the USB connection.
Bug: http://b/27297963
Change-Id: Ie6180d0b59d133120c5755e239e76ab33ed3cc1d
Diffstat (limited to 'transport.h')
-rw-r--r-- | transport.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/transport.h b/transport.h index 76d6afa..4c0c008 100644 --- a/transport.h +++ b/transport.h @@ -50,7 +50,6 @@ public: // it's better to do this piece by piece. atransport() { - auth_fde = {}; transport_fde = {}; protocol_version = A_VERSION; max_payload = MAX_PAYLOAD; @@ -87,7 +86,6 @@ public: void* key = nullptr; unsigned char token[TOKEN_SIZE] = {}; - fdevent auth_fde; size_t failed_auth_attempts = 0; const std::string connection_state_name() const; |