asgardius/virtualx-engine
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Packages Wiki Activity Actions

Fix heap-use-after-free when using TileMap editor

Browse source
This commit is contained in:
Haoyu Qiu 2022-12-30 14:15:30 +08:00
parent 0daa86d70e
commit 1ac35c38be
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
editor/plugins/tiles/tile_map_editor.cpp
View file

@ -1488,13 +1488,15 @@ void TileMapEditorTilesPlugin::_update_fix_selected_and_hovered() {
} }
// Selection if needed. // Selection if needed.
for (RBSet<TileMapCell>::Element *E = tile_set_selection.front(); E; E = E->next()) { for (RBSet<TileMapCell>::Element *E = tile_set_selection.front(); E;) {
RBSet<TileMapCell>::Element *N = E->next();
const TileMapCell *selected = &(E->get()); const TileMapCell *selected = &(E->get());
if (!tile_set->has_source(selected->source_id) || if (!tile_set->has_source(selected->source_id) ||
!tile_set->get_source(selected->source_id)->has_tile(selected->get_atlas_coords()) || !tile_set->get_source(selected->source_id)->has_tile(selected->get_atlas_coords()) ||
!tile_set->get_source(selected->source_id)->has_alternative_tile(selected->get_atlas_coords(), selected->alternative_tile)) { !tile_set->get_source(selected->source_id)->has_alternative_tile(selected->get_atlas_coords(), selected->alternative_tile)) {
tile_set_selection.erase(E); tile_set_selection.erase(E);
} }
E = N;
} }
if (!tile_map_selection.is_empty()) { if (!tile_map_selection.is_empty()) {