archive/blender-archive
Archived
1
1
Fork 0
Code Pull Requests Packages Activity

BGE correction: AddRef() on source object of property actuator to protect against crash if the source object is deleted (bad game design anyway)

Browse Source
This commit is contained in:
Benoit Bolsee
2008-03-22 14:47:42 +00:00
parent 1d0e1d5406
commit 6615f8b74f
2 changed files with 9 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
projectfiles_vc7/blender/src/BL_src.vcproj
View File

@@ -547,6 +547,9 @@
<File
RelativePath="..\..\..\source\blender\src\transform_manipulator.c">
</File>
<File
RelativePath="..\..\..\source\blender\src\transform_ndofinput.c">
</File>
<File
RelativePath="..\..\..\source\blender\src\transform_numinput.c">
</File>

6
source/gameengine/GameLogic/SCA_PropertyActuator.cpp
View File

@@ -52,10 +52,16 @@ SCA_PropertyActuator::SCA_PropertyActuator(SCA_IObject* gameobj,CValue* sourceOb
m_exprtxt(expr),
m_sourceObj(sourceObj)
{
// protect ourselves against someone else deleting the source object
// don't protect against ourselves: it would create a dead lock
if (m_sourceObj && m_sourceObj != GetParent())
m_sourceObj->AddRef();
}
SCA_PropertyActuator::~SCA_PropertyActuator()
{
if (m_sourceObj && m_sourceObj != GetParent())
m_sourceObj->Release();
}
bool SCA_PropertyActuator::Update()