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

Fix T91123: Freeing meshes in edit-mode leaks memory

Browse Source 
Freeing the edit-mesh pointer wasn't free the edit-mesh data it's self.

Unlinking from the outliner or through the Python API leaked memory.

This didn't often cause leaks in practice since ED_editors_exit
exits edit-mode for all objects.

Now freeing mesh data frees the edit-mode data as well,
this matches the behavior of other object types that support edit-mode.
This commit is contained in:
Campbell Barton
2021-09-02 15:29:34 +10:00
parent 809b33b69a
commit 0708733c46
2 changed files with 12 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
source/blender/blenkernel/intern/editmesh.c
View File

@@ -211,10 +211,14 @@ void BKE_editmesh_looptri_and_normals_calc_with_partial(BMEditMesh *em,
void BKE_editmesh_free_derived_caches(BMEditMesh *em)
{
if (em->mesh_eval_cage) {
BKE_id_free(NULL, em->mesh_eval_cage);
Mesh *me = em->mesh_eval_cage;
MEM_SAFE_FREE(me->edit_mesh);
BKE_id_free(NULL, me);
}
if (em->mesh_eval_final && em->mesh_eval_final != em->mesh_eval_cage) {
BKE_id_free(NULL, em->mesh_eval_final);
Mesh *me = em->mesh_eval_final;
MEM_SAFE_FREE(me->edit_mesh);
BKE_id_free(NULL, me);
}
em->mesh_eval_cage = em->mesh_eval_final = NULL;

6
source/blender/blenkernel/intern/mesh.c
View File

@@ -160,6 +160,12 @@ static void mesh_free_data(ID *id)
BLI_freelistN(&mesh->vertex_group_names);
if (mesh->edit_mesh) {
BKE_editmesh_free_data(mesh->edit_mesh);
MEM_freeN(mesh->edit_mesh);
mesh->edit_mesh = NULL;
}
BKE_mesh_runtime_clear_cache(mesh);
mesh_clear_geometry(mesh);
MEM_SAFE_FREE(mesh->mat);