commit c524813b806a64123d376d1afb4d21b09a6de453
author Yin-Chia Yeh <yinchiayeh@google.com> Wed Aug 15 12:19:20 2018 -0700
committer Yin-Chia Yeh <yinchiayeh@google.com> Wed Aug 15 15:08:36 2018 -0700
tree f94bbd0c687bf3eb4dab26c97eedd49419bfec71
parent c5be8b200a5927d8d9e89f425ca1e544884b6031

Camera: fix race issue in Camera2 clients

Updating of the mDevice sp<> is racy.
This change will cause the Camera3Device object lives until
the client is destructed (which can be a few seconds after
camera app closed and GC kicking in to delete the client)
The benefit is the dumpDevice can remain lock free.

Also fix a problem where we call virtual function disconnect in
Camera3Device destructor.

Test: the double free problem cannot reproduce
Bug: 112639939
Change-Id: I77762db8f59cf33891631d13fc3bdb3830ae08fb