summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVishnu Nair <vishnun@google.com>2020-05-06 01:36:37 +0000
committerAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>2020-05-06 01:36:37 +0000
commitf5d934e7ba225f3def682811e9cdc0a13062cd2f (patch)
treef345be860c0d16ad43ad67a8721ced87f33e0395
parentaf8f6cf3aba3398b36117dd3acce6f5a3c0610c6 (diff)
parenta03e1302db3c51be1e453af179e5549468b501ab (diff)
downloadnative-f5d934e7ba225f3def682811e9cdc0a13062cd2f.tar.gz
Remove unsafe cast in Client::createWithSurfaceParent am: a03e1302db
Change-Id: I5b890ef48b214ed9702acd7a2cac52162a9c0ddb
-rw-r--r--services/surfaceflinger/Client.cpp5
1 files changed, 0 insertions, 5 deletions
diff --git a/services/surfaceflinger/Client.cpp b/services/surfaceflinger/Client.cpp
index 6bfd302b5c..c526f7f18f 100644
--- a/services/surfaceflinger/Client.cpp
+++ b/services/surfaceflinger/Client.cpp
@@ -88,11 +88,6 @@ status_t Client::createWithSurfaceParent(const String8& name, uint32_t w, uint32
sp<IGraphicBufferProducer>* gbp) {
if (mFlinger->authenticateSurfaceTexture(parent) == false) {
ALOGE("failed to authenticate surface texture");
- // The extra parent layer check below before returning is to help with debugging
- // b/134888387. Once the bug is fixed the check can be deleted.
- if ((static_cast<MonitoredProducer*>(parent.get()))->getLayer() == nullptr) {
- ALOGE("failed to find parent layer");
- }
return BAD_VALUE;
}