tag a87675806684a3ee9de0f28e3a5cc4f02b25faee
tagger The Android Open Source Project <initial-contribution@android.com> Tue Sep 08 15:10:56 2020 -0700
object a9aa20052b230c698b91b93b616f2e39e5f7f610

Android 8.0.0 release 50

commit a9aa20052b230c698b91b93b616f2e39e5f7f610
author Andrew Lewis <andrewlewis@google.com> Wed May 20 16:49:23 2020 +0100
committer Anis Assi <anisassi@google.com> Thu Jun 04 11:26:21 2020 -0700
tree 127e7e637d90b1536f85a0f1b52f795b30526833
parent 4748c54e106038d0442e7c5f3c473e15f839391f

Fix potential use-after-free issue

NuPlayerListener passed this out of its constructor to
IStreamSource->setListener, and it would get wrapped in a smart pointer,
then free'd when that method returned.

Move the setListener call after instantiation of NuPlayerListener to
avoid passing this out of the NuPlayerListener constructor.

Test: tested in presubmit
Bug: 151456667
Change-Id: I996d9ad9eaf0e52992b7f9e10fd94a94c350ad73
(cherry picked from commit aad519df7fe3b69c8247669ea77ce339a44e8c51)
(cherry picked from commit 6aaef5a99f15da847a8c55a137bfc6263829e8ff)