diff options
| author | 2022-06-29 22:13:49 +0000 | |
|---|---|---|
| committer | 2022-06-29 22:14:26 +0000 | |
| commit | 728dba1f95927d8f7acb52d09c65c486f81ea50d (patch) | |
| tree | 6af44e4169ebd158a23fe1eb4124aaa99fbc4d45 /libs/gui/BLASTBufferQueue.cpp | |
| parent | 715b4413b93d90c9d2a1f559c9b0fac69dd09f4f (diff) | |
BBQ: Remove BufferFreedListener
If BBQ is destroyed without calling abandon, its BufferItemConsumer
will call onBufferFreed. Since BBQ registers itself as the
BufferFreedListener we end up trying to promote and destroy the
BBQ again. The BufferFreedListener is not used so as a simple fix,
we remove the listener.
Test: presubmit
Bug: 200246498
Change-Id: I7b2be265b3c09d691cc0c41b80c73067f9a8b84d
Diffstat (limited to 'libs/gui/BLASTBufferQueue.cpp')
| -rw-r--r-- | libs/gui/BLASTBufferQueue.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/libs/gui/BLASTBufferQueue.cpp b/libs/gui/BLASTBufferQueue.cpp index f34061492a..77022dc9b8 100644 --- a/libs/gui/BLASTBufferQueue.cpp +++ b/libs/gui/BLASTBufferQueue.cpp @@ -159,7 +159,6 @@ BLASTBufferQueue::BLASTBufferQueue(const std::string& name, bool updateDestinati id++; mBufferItemConsumer->setName(String8(consumerName.c_str())); mBufferItemConsumer->setFrameAvailableListener(this); - mBufferItemConsumer->setBufferFreedListener(this); ComposerServiceAIDL::getComposerService()->getMaxAcquiredBufferCount(&mMaxAcquiredBuffers); mBufferItemConsumer->setMaxAcquiredBufferCount(mMaxAcquiredBuffers); @@ -1114,7 +1113,6 @@ void BLASTBufferQueue::abandon() { if (mBufferItemConsumer != nullptr) { mBufferItemConsumer->abandon(); mBufferItemConsumer->setFrameAvailableListener(nullptr); - mBufferItemConsumer->setBufferFreedListener(nullptr); } mBufferItemConsumer = nullptr; mConsumer = nullptr; |