ilusha/blender
1
0
Fork 0
mirror of https://github.com/blender/blender synced 2026-09-29 04:37:17 +03:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix T70965: Undo crash with specific file

Browse source 
Not sure how the WorkSpaceLayout.screen pointer could end up being NULL,
but apparently that happens, or at least happened with older files.

Rather than just adding NULL-checks, prefer not keeping around those
invalid layouts at all.
This commit is contained in:
Julian Eisel 2020-01-15 17:46:50 +01:00
parent c27acbcfb7
commit ca49643f3c
2 changed files with 11 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

8
source/blender/blenkernel/intern/workspace.c
View file

@ -247,8 +247,12 @@ WorkSpaceLayout *BKE_workspace_layout_add(Main *bmain,
void BKE_workspace_layout_remove(Main *bmain, WorkSpace *workspace, WorkSpaceLayout *layout)
{
id_us_min(&layout->screen->id);
BKE_id_free(bmain, layout->screen);
/* Screen should usually be set, but we call this from file reading to get rid of invalid
* layouts. */
if (layout->screen) {
id_us_min(&layout->screen->id);
BKE_id_free(bmain, layout->screen);
}
BLI_freelinkN(&workspace->layouts, layout);
}

5
source/blender/blenloader/intern/readfile.c
View file

@ -3377,6 +3377,11 @@ static void lib_link_workspaces(FileData *fd, Main *bmain)
}
}
}
else {
/* If we're reading a layout without screen stored, it's useless and we shouldn't keep it
* around. */
BKE_workspace_layout_remove(bmain, workspace, layout);
}
}
id->tag &= ~LIB_TAG_NEED_LINK;