From 16c75cf594a92b232e46f922d644ac0a70088bad Mon Sep 17 00:00:00 2001 From: Mitchell Stokes Date: Mon, 17 Feb 2014 14:35:09 -0800 Subject: Fix T38654: Using a non-camera object for the SetCamera Scene Actuator causes a segfault. Non-camera objects are not supported as cameras, and trying to use them just causes memory errors (some bad typecasts are involved). When converting a Scene Actuator, only use the object as a camera if it is actually a camera. Also, the logic brick UI now filters for camera objects, which should help avoid confusion. --- source/gameengine/Converter/KX_ConvertActuators.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'source/gameengine') diff --git a/source/gameengine/Converter/KX_ConvertActuators.cpp b/source/gameengine/Converter/KX_ConvertActuators.cpp index 26401fcd868..725e9815a67 100644 --- a/source/gameengine/Converter/KX_ConvertActuators.cpp +++ b/source/gameengine/Converter/KX_ConvertActuators.cpp @@ -758,7 +758,9 @@ void BL_ConvertActuators(const char* maggiename, mode = KX_SceneActuator::KX_SCENE_SET_CAMERA; if (sceneact->camera) { - cam = (KX_Camera*) converter->FindGameObject(sceneact->camera); + KX_GameObject *tmp = converter->FindGameObject(sceneact->camera); + if (tmp && tmp->GetGameObjectType() == SCA_IObject::OBJ_CAMERA) + cam = (KX_Camera*)tmp; } break; case ACT_SCENE_RESTART: -- cgit v1.2.3