[Bf-blender-cvs] [5d67b524412] blender-v3.3-release: Fix T100562: Realize Instances node crashes when there is an attribute name collision

Jacques Lucke noreply at git.blender.org
Tue Aug 23 10:57:35 CEST 2022


Commit: 5d67b524412d0dab936170eef65c508930ba4c30
Author: Jacques Lucke
Date:   Tue Aug 23 10:57:28 2022 +0200
Branches: blender-v3.3-release
https://developer.blender.org/rB5d67b524412d0dab936170eef65c508930ba4c30

Fix T100562: Realize Instances node crashes when there is an attribute name collision

===================================================================

M	source/blender/geometry/intern/realize_instances.cc

===================================================================

diff --git a/source/blender/geometry/intern/realize_instances.cc b/source/blender/geometry/intern/realize_instances.cc
index 4b3b184536b..87a610ac0a2 100644
--- a/source/blender/geometry/intern/realize_instances.cc
+++ b/source/blender/geometry/intern/realize_instances.cc
@@ -14,6 +14,7 @@
 
 #include "BKE_collection.h"
 #include "BKE_curves.hh"
+#include "BKE_deform.h"
 #include "BKE_geometry_set_instances.hh"
 #include "BKE_material.h"
 #include "BKE_mesh.h"
@@ -997,6 +998,9 @@ static void execute_realize_mesh_tasks(const RealizeInstancesOptions &options,
   const RealizeMeshTask &first_task = tasks.first();
   const Mesh &first_mesh = *first_task.mesh_info->mesh;
   BKE_mesh_copy_parameters_for_eval(dst_mesh, &first_mesh);
+  /* The above line also copies vertex group names. We don't want that here because the new
+   * attributes are added explicitly below. */
+  BLI_freelistN(&dst_mesh->vertex_group_names);
 
   /* Add materials. */
   for (const int i : IndexRange(ordered_materials.size())) {



More information about the Bf-blender-cvs mailing list