diff options
author | Campbell Barton <ideasman42@gmail.com> | 2021-09-02 08:29:34 +0300 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2021-09-02 08:30:54 +0300 |
commit | 0708733c46cb6b50697c1b80a6541d6fb410267e (patch) | |
tree | 2ddb2c60366efa656f0207e8463a09f2cfae6ad5 /source/blender/blenkernel/intern/editmesh.c | |
parent | 809b33b69a04fc37e1535697b667158ca80f6d60 (diff) |
Fix T91123: Freeing meshes in edit-mode leaks memory
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.
Diffstat (limited to 'source/blender/blenkernel/intern/editmesh.c')
-rw-r--r-- | source/blender/blenkernel/intern/editmesh.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/source/blender/blenkernel/intern/editmesh.c b/source/blender/blenkernel/intern/editmesh.c index 83e03ef44f5..a925dfa4da8 100644 --- a/source/blender/blenkernel/intern/editmesh.c +++ b/source/blender/blenkernel/intern/editmesh.c @@ -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; |