Bladeren bron

Merge pull request #53810 from groud/fix_crash

Fixes crash in TileSetAtlasSource::get_tiles_to_be_removed_on_change
Rémi Verschelde 3 jaren geleden
bovenliggende
commit
668ceda9cb
1 gewijzigde bestanden met toevoegingen van 4 en 0 verwijderingen
  1. 4 0
      scene/resources/tile_set.cpp

+ 4 - 0
scene/resources/tile_set.cpp

@@ -3565,6 +3565,10 @@ bool TileSetAtlasSource::has_room_for_tile(Vector2i p_atlas_coords, Vector2i p_s
 }
 
 PackedVector2Array TileSetAtlasSource::get_tiles_to_be_removed_on_change(Ref<Texture2D> p_texture, Vector2i p_margins, Vector2i p_separation, Vector2i p_texture_region_size) {
+	ERR_FAIL_COND_V(p_margins.x < 0 || p_margins.y < 0, PackedVector2Array());
+	ERR_FAIL_COND_V(p_separation.x < 0 || p_separation.y < 0, PackedVector2Array());
+	ERR_FAIL_COND_V(p_texture_region_size.x <= 0 || p_texture_region_size.y <= 0, PackedVector2Array());
+
 	// Compute the new atlas grid size.
 	Size2 new_grid_size;
 	if (p_texture.is_valid()) {