From 32c2cba4ef1c940dab559cdc47b886c60cc9f2e1 Mon Sep 17 00:00:00 2001 From: Sergey Sharybin Date: Thu, 20 Oct 2016 13:32:52 +0200 Subject: Fix T49797: Exception from scene update handler might leak external engine descriptors This was causing memory leaks in Cycles. Some more detailed information in the comment in the code. Seems to be safe and nice enough for 2.78a. --- source/blender/python/intern/bpy_app_handlers.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) (limited to 'source/blender/python/intern/bpy_app_handlers.c') diff --git a/source/blender/python/intern/bpy_app_handlers.c b/source/blender/python/intern/bpy_app_handlers.c index 6a8b0b065c2..1cc2d6f1307 100644 --- a/source/blender/python/intern/bpy_app_handlers.c +++ b/source/blender/python/intern/bpy_app_handlers.c @@ -317,7 +317,13 @@ void bpy_app_generic_callback(struct Main *UNUSED(main), struct ID *id, void *ar func = PyList_GET_ITEM(cb_list, pos); ret = PyObject_Call(func, args, NULL); if (ret == NULL) { - PyErr_Print(); + /* Don't set last system variables because they might cause some + * dangling pointers to external render engines (when exception + * happens during rendering) which will break logic of render pipeline + * which expects to be the only user of render engine when rendering + * is finished. + */ + PyErr_PrintEx(0); PyErr_Clear(); } else { -- cgit v1.2.3