summaryrefslogtreecommitdiff
path: root/services/surfaceflinger/LayerBase.cpp
diff options
context:
space:
mode:
author Simon Wilson <simonwilson@google.com> 2011-05-24 17:07:39 -0700
committer Android (Google) Code Review <android-gerrit@google.com> 2011-05-24 17:07:39 -0700
commitc9cd2387b6938a6fbefc731d2177902266f2a130 (patch)
treeb6b7f4ca61ee3c9d29e1c2d1ded10993e4cca4a0 /services/surfaceflinger/LayerBase.cpp
parent34f5991ebe8b203f1720caf95889c15dd131d86e (diff)
parent6f96080f333c70104424f42ac263c4c9c7a9dd2a (diff)
Merge changes I37f0f315,I8cbf6044,Ibb598931,I5262bf11 into gingerbread
* changes: Fix a race that could cause GL commands to be executed from the wrong thread. RefBase subclasses can now decide how they want to be destroyed. Fix a race in SurfaceFlinger that could cause layers to be leaked forever. Fix a race-condtion in SurfaceFlinger that could lead to a crash.
Diffstat (limited to 'services/surfaceflinger/LayerBase.cpp')
-rw-r--r--services/surfaceflinger/LayerBase.cpp5
1 files changed, 1 insertions, 4 deletions
diff --git a/services/surfaceflinger/LayerBase.cpp b/services/surfaceflinger/LayerBase.cpp
index 21c36e18485e..3986fdef0f30 100644
--- a/services/surfaceflinger/LayerBase.cpp
+++ b/services/surfaceflinger/LayerBase.cpp
@@ -583,10 +583,7 @@ LayerBaseClient::Surface::~Surface()
*/
// destroy client resources
- sp<LayerBaseClient> layer = getOwner();
- if (layer != 0) {
- mFlinger->destroySurface(layer);
- }
+ mFlinger->destroySurface(mOwner);
}
sp<LayerBaseClient> LayerBaseClient::Surface::getOwner() const {