tree 3d0993a74bf5c7141307c61553ceacf87a3b62c1
parent d474283388cd7a22241185a8a72576be415d4295
author Garrick Evans <garrick@google.com> 1558055978 +0900
committer chrome-bot <chrome-bot@chromium.org> 1558633165 -0700

arc: networkd: Fix possible race in adb-proxy.

After a connection is established to adbd and the socket forwarder starts,
the proxy cleans up any defunct forwarders. If this code runs before the
forwarder thread actually starts then the one that was just creatd will be
deleted. Since the forwarder joins the thread in the destructor, this can
lead to a deadlock (the thread will never start so join will never return).

BUG=b:132213940
TEST=betty-paladin-tryjob currently in-flight (cannot repro locally)

Change-Id: I6193c6303c19f5e600c3c95a2ce33ffd9687e472
Reviewed-on: https://chromium-review.googlesource.com/1615407
Commit-Ready: Garrick Evans <garrick@chromium.org>
Tested-by: Garrick Evans <garrick@chromium.org>
Legacy-Commit-Queue: Commit Bot <commit-bot@chromium.org>
Reviewed-by: Hugo Benichi <hugobenichi@google.com>