Page MenuHome
Create Task
Maniphest T72887

Crash when issuing an undo operation immediately after linking a collection when an outliner's display mode is set to "Blender File"
Closed, ResolvedPublicBUG
Actions

Description

System Information
Operating system: Linux-5.3.0-26-generic-x86_64-with-Ubuntu-19.10-eoan 64 Bits
Graphics card: AMD Radeon RX 5700 XT (NAVI10, DRM 3.33.0, 5.3.0-26-generic, LLVM 9.0.0) X.Org 4.5 (Core Profile) Mesa 19.2.1

Blender Version
Broken: version: 2.82 (sub 6), branch: master, commit date: 2020-01-03 10:02, hash: rB9b74e648c512
Worked: Never

Short description of error
Blender crashes when issuing an undo operation immediately after linking a collection when an Outliner's display mode set to Blender File
It doesn't matter how many Outliners are open, as long as one is set to Blender File.
This only happens when linking, not appending. No other data blocks causes the crash except collections.

Log files:

blender.crash.txt887 BDownload

terminal-output.txt1 KBDownload

Exact steps for others to reproduce the error
Apparently the issue occurs only if the linked collection also has linked stuff? In the test case, the collection to be linked has linked materials.
Blender does not crash when linking standalone collections.

Download these files:

TEST.blend545 KBDownload

1.blend543 KBDownload

2.blend546 KBDownload

  1. Open TEST.blend
  2. Link a collection from 2.blend (2.blend has an object linked from 1.blend; now the Outliner has 2 linked files)
  3. Undo the operation (Ctrl+Z)

Revisions and Commits

rB Blender
rBd74f9c4b7b4f Fix T72887: Outliner crashes when after undo of some linking operation.

Related Objects

Event Timeline

ronsn (ronsn) added a subscriber: ronsn (ronsn).Fri, Jan 3, 7:49 PM

I can not reproduce this issue.

–––––––––––––––––––––––––

System Information
Operating system: Linux-5.0.0-37-generic-x86_64-with-debian-buster-sid 64 Bits (Ubuntu 18.04)
Graphics card: GeForce GTX 1660 Ti/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 435.21

Blender Version
version: 2.82 (sub 6), branch: master, commit date: 2020-01-03 10:02, hash: rB9b74e648c512

Robert Guetzkow (rjg) changed the task status from Needs Triage to Needs Information from User.Fri, Jan 3, 8:20 PM
Robert Guetzkow (rjg) added a subscriber: Robert Guetzkow (rjg).

I cannot reproduce this on Windows with the latest master either. Is there anything special about the collection you're linking?

Rey Leonard M. Amorato (rl.amorato) changed the task status from Needs Information from User to Needs Triage.Fri, Jan 3, 8:34 PM
Rey Leonard M. Amorato (rl.amorato) updated the task description. (Show Details)
Rey Leonard M. Amorato (rl.amorato) added a comment.Fri, Jan 3, 8:38 PM

Added more info; the collection to be linked also has linked materials.

ronsn (ronsn) added a comment.Fri, Jan 3, 8:55 PM

Okay, I can confirm that issue, that Blender crashes if you link something from a file where also something is linked in!

ronsn (ronsn) changed the task status from Needs Triage to Confirmed.Fri, Jan 3, 8:55 PM
ronsn (ronsn) added a comment.Fri, Jan 3, 10:52 PM

This issue also occurs in version: 2.80 (sub 75), branch: master, commit date: 2019-07-29 14:47, hash: rBf6cb5f54494e

@Robert Guetzkow (rjg): This task might have a High priority, isn't it?

Robert Guetzkow (rjg) triaged this task as High priority.EditedFri, Jan 3, 11:36 PM

I can confirm this as well. Since this bug is crashing Blender, I'm setting the priority to high.

The tree elements of the outliner contain invalid pointers.

BLI_addtail(ListBase * listbase, void * vlink) Line 119	C
outliner_add_element(SpaceOutliner * soops, ListBase * lb, void * idv, TreeElement * parent, short type, short index) Line 796	C
outliner_add_library_contents(Main * mainvar, SpaceOutliner * soops, ListBase * lb, Library * lib) Line 1282	C
outliner_build_tree(Main * mainvar, Scene * scene, ViewLayer * view_layer, SpaceOutliner * soops, ARegion * ar) Line 2329	C
draw_outliner(const bContext * C) Line 3635	C
outliner_main_region_draw(const bContext * C, ARegion * ar) Line 96	C
ED_region_do_draw(bContext * C, ARegion * ar) Line 539	C
wm_draw_window_offscreen(bContext * C, wmWindow * win, bool stereo) Line 628	C
wm_draw_window(bContext * C, wmWindow * win) Line 766	C
wm_draw_update(bContext * C) Line 946	C
WM_main(bContext * C) Line 424	C
main(int argc, const unsigned char * * UNUSED_argv_c) Line 520	C
Germano Cavalcante (mano-wii) updated the task description. (Show Details)Thu, Jan 9, 7:55 PM
Germano Cavalcante (mano-wii) added a project: Data, Assets & I/O.
Germano Cavalcante (mano-wii) changed the subtype of this task from "Report" to "Bug".
Bastien Montagne (mont29) lowered the priority of this task from High to Normal.Thu, Jan 16, 9:42 AM
Bastien Montagne (mont29) claimed this task.Thu, Jan 16, 12:01 PM
Bastien Montagne (mont29) added a subscriber: Bastien Montagne (mont29).

This actually exhibits two issues:

  1. Outliner crash (looks like it does not like the situation generated by undo in that case).
  2. Undo code keeps reference to unused libraries, and even unused indirectly linked datablocks.

Will try to fix the crash in outliner first, then we'll see about cleaning up again that black dark magic of library handling in undos…