summaryrefslogtreecommitdiffstats
path: root/libs
diff options
context:
space:
mode:
authorChris Craik <ccraik@google.com>2014-04-11 22:46:20 +0000
committerAndroid (Google) Code Review <android-gerrit@google.com>2014-04-11 22:46:20 +0000
commit286ffe4b28d2c004ce98f7ddb3a5f34441ab0d80 (patch)
treeba94b0e5379e56f60ae5802169fdc50fd0ec4c62 /libs
parentdd35634cb0059b4b745161eb80234d8e8da38033 (diff)
parent143912fef8eff58146705849a0ba441ab6163409 (diff)
downloadframeworks_base-286ffe4b28d2c004ce98f7ddb3a5f34441ab0d80.zip
frameworks_base-286ffe4b28d2c004ce98f7ddb3a5f34441ab0d80.tar.gz
frameworks_base-286ffe4b28d2c004ce98f7ddb3a5f34441ab0d80.tar.bz2
Merge "Remove use-after-free signal in RenderNode"
Diffstat (limited to 'libs')
-rw-r--r--libs/hwui/RenderNode.cpp10
-rw-r--r--libs/hwui/RenderNode.h1
2 files changed, 1 insertions, 10 deletions
diff --git a/libs/hwui/RenderNode.cpp b/libs/hwui/RenderNode.cpp
index 2489c92..c55ebd6 100644
--- a/libs/hwui/RenderNode.cpp
+++ b/libs/hwui/RenderNode.cpp
@@ -50,17 +50,13 @@ void RenderNode::outputLogBuffer(int fd) {
}
RenderNode::RenderNode()
- : mDestroyed(false)
- , mNeedsPropertiesSync(false)
+ : mNeedsPropertiesSync(false)
, mNeedsDisplayListDataSync(false)
, mDisplayListData(0)
, mStagingDisplayListData(0) {
}
RenderNode::~RenderNode() {
- LOG_ALWAYS_FATAL_IF(mDestroyed, "Double destroyed DisplayList %p", this);
-
- mDestroyed = true;
delete mDisplayListData;
delete mStagingDisplayListData;
}
@@ -525,10 +521,6 @@ void RenderNode::issueOperationsOfProjectedChildren(OpenGLRenderer& renderer, T&
template <class T>
void RenderNode::issueOperations(OpenGLRenderer& renderer, T& handler) {
const int level = handler.level();
- if (CC_UNLIKELY(mDestroyed)) { // temporary debug logging
- ALOGW("Error: %s is drawing after destruction", mName.string());
- CRASH();
- }
if (mDisplayListData->isEmpty() || properties().getAlpha() <= 0) {
DISPLAY_LIST_LOGD("%*sEmpty display list (%p, %s)", level * 2, "", this, mName.string());
return;
diff --git a/libs/hwui/RenderNode.h b/libs/hwui/RenderNode.h
index 7853701..9e6ee3f 100644
--- a/libs/hwui/RenderNode.h
+++ b/libs/hwui/RenderNode.h
@@ -210,7 +210,6 @@ private:
void prepareSubTree(TreeInfo& info, DisplayListData* subtree);
String8 mName;
- bool mDestroyed; // used for debugging crash, TODO: remove once invalid state crash fixed
bool mNeedsPropertiesSync;
RenderProperties mProperties;