Fix bmesh_to_mesh freeing possibly referenced vertices
authorSergey Sharybin <sergey.vfx@gmail.com>
Fri, 31 May 2019 13:06:01 +0000 (15:06 +0200)
committerSergey Sharybin <sergey.vfx@gmail.com>
Fri, 31 May 2019 13:07:15 +0000 (15:07 +0200)
Fixes Godot exporter issue reported in T65285.

source/blender/bmesh/intern/bmesh_mesh_conv.c
source/blender/python/bmesh/bmesh_py_types.c

index 0ea3304..7a10016 100644 (file)
@@ -611,12 +611,7 @@ void BM_mesh_bm_to_me(Main *bmain, BMesh *bm, Mesh *me, const struct BMeshToMesh
 
   /* lets save the old verts just in case we are actually working on
    * a key ... we now do processing of the keys at the end */
-  oldverts = me->mvert;
-
-  /* don't free this yet */
-  if (oldverts) {
-    CustomData_set_layer(&me->vdata, CD_MVERT, NULL);
-  }
+  oldverts = MEM_dupallocN(me->mvert);
 
   /* free custom data */
   CustomData_free(&me->vdata, me->totvert);
@@ -975,7 +970,7 @@ void BM_mesh_bm_to_me(Main *bmain, BMesh *bm, Mesh *me, const struct BMeshToMesh
     }
   }
 
-  if (oldverts) {
+  if (oldverts != NULL) {
     MEM_freeN(oldverts);
   }
 
index eb611e6..89d15ed 100644 (file)
@@ -31,6 +31,7 @@
 #include "BKE_customdata.h"
 #include "BKE_global.h"
 #include "BKE_library.h"
+#include "BKE_mesh.h"
 #include "BKE_mesh_runtime.h"
 
 #include "DEG_depsgraph.h"
@@ -1089,13 +1090,19 @@ static PyObject *bpy_bmesh_to_mesh(BPy_BMesh *self, PyObject *args)
 
   bm = self->bm;
 
-  BLI_assert(BKE_id_is_in_global_main(&me->id));
-  BM_mesh_bm_to_me(G_MAIN, /* XXX UGLY! */
-                   bm,
-                   me,
-                   (&(struct BMeshToMeshParams){
-                       .calc_object_remap = true,
-                   }));
+  struct Main *bmain = NULL;
+  struct BMeshToMeshParams params = {0};
+  if (me->id.tag & LIB_TAG_NO_MAIN) {
+    /* Mesh might be coming from a self-contained source like object.to_mesh(). No need to remap
+     * anything in this case. */
+  }
+  else {
+    BLI_assert(BKE_id_is_in_global_main(&me->id));
+    bmain = G_MAIN; /* XXX UGLY! */
+    params.calc_object_remap = true;
+  }
+
+  BM_mesh_bm_to_me(bmain, bm, me, &params);
 
   /* we could have the user do this but if they forget blender can easy crash
    * since the references arrays for the objects derived meshes are now invalid */