From 9edd645638205c6c2bda7f3ca0b0853700a99f79 Mon Sep 17 00:00:00 2001 From: Hans Goudey Date: Mon, 12 May 2025 23:25:21 -0400 Subject: [PATCH] Fix #138790: Crash applying empty geometry node modifier on point cloud Copying the point cloud is basically free in the scheme of things. Better to do that than check if the output point cloud is the same. --- source/blender/editors/object/object_modifier.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/source/blender/editors/object/object_modifier.cc b/source/blender/editors/object/object_modifier.cc index f0f826c732e..0d7bb5b5c80 100644 --- a/source/blender/editors/object/object_modifier.cc +++ b/source/blender/editors/object/object_modifier.cc @@ -1203,7 +1203,7 @@ static bool modifier_apply_obdata(ReportList *reports, } bke::GeometrySet geometry_set = bke::GeometrySet::from_pointcloud( - &points, bke::GeometryOwnershipType::ReadOnly); + BKE_pointcloud_copy_for_eval(&points)); ModifierEvalContext mectx = {depsgraph, ob, MOD_APPLY_TO_ORIGINAL}; mti->modify_geometry_set(md_eval, &mectx, &geometry_set);