bartvdbraak/blender
1
0
Fork 4
Code Issues 4 Pull Requests 1 Actions 1 Packages Projects Releases Wiki Activity

Fix #110485: Mesh to curve node incorrect implicit sharing crash

Browse Source 
Regression caused by 301f13191befb727fe96e8d919798eb07077c017.
To share attribute arrays, domain sizes should be the same.

Pull Request: https://projects.blender.org/blender/blender/pulls/110487
This commit is contained in:
Iliya Katueshenock 2023-08-01 03:53:01 +02:00 committed by Hans Goudey
parent 61ee554e3f
commit 803589ca5a
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
source/blender/geometry/intern/mesh_to_curve_convert.cc
View File

@ -58,7 +58,9 @@ BLI_NOINLINE bke::CurvesGeometry create_curve_from_vert_indices(
curves.cyclic_for_write().slice(cyclic_curves).fill(true);
}
const bool share_vert_data = indices_are_full_ordered_copy(vert_indices);
const int src_total_points = mesh_attributes.domain_size(ATTR_DOMAIN_POINT);
const bool share_vert_data = vert_indices.size() == src_total_points &&
indices_are_full_ordered_copy(vert_indices);
if (share_vert_data) {
bke::copy_attributes(
mesh_attributes, ATTR_DOMAIN_POINT, propagation_info, {}, curves_attributes);