tag 8cd8212d82c1b4e1e3ae98b509d072bcba9ac197
tagger The Android Open Source Project <initial-contribution@android.com> Fri Dec 01 13:57:46 2017 -0800
object fc73edfdc7e025de79ea6d2ade4fb0e1ce9bbef7

Android O MR1 Preview 1

commit fc73edfdc7e025de79ea6d2ade4fb0e1ce9bbef7
author Yifan Hong <elsk@google.com> Tue Aug 29 11:39:07 2017 -0700
committer Yifan Hong <elsk@google.com> Tue Aug 29 22:41:14 2017 +0000
tree cb60077e5d52e55ba9c0964f83983eb74a6e3468
parent 48602dfa0ef4a40b58c6376bab82c6e46d6f6a0b

Fix ownership of objects returned by VintfObject::Get*

VintfObject::Get* is not thread-safe; the raw pointers
returned to clients can be invalidated by another thread
that calls the same VintfObject::Get* with skipCache = true.

Fix this by sharing ownership to callers as well.
VintfObject::Get* now returns a shared_ptr, and clients
can destroy the shared_ptr once they finish reading it.

Test: libvintf_test
Test: vintf_object_test
Test: `adb shell vintf` shows compatible

Bug: 65166415

Change-Id: I931cf0864a8dba3d83fe41f715b94e1a21b266eb