Crash when appending object using operator called from template icon view, followed by undo #80217

New Issue

MACHIN3 · 2020-08-29T13:22:46+02:00

MACHIN3 commented

2020-08-29 13:22:46 +02:00

System Information
Operating system: Linux-4.15.0-112-generic-x86_64-with-debian-buster-sid 64 Bits
Graphics card: GeForce GTX 1050/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 435.21

Blender Version
Broken: version: 2.83.5, branch: master, commit date: 2020-08-19 06:07, hash: c2b144df39, 2.90 is also affected

Short description of error
I have an operator that appends an object from another file. Calling it and undoing right after, works as expected unless that same operator is called from an update function of an enum, storing preview collection items, as is done for template icon views. Blender will crash on 2.83.5 and 2.90.

append_obj_undo_crash_from_template_icon_view.zip

video demo

Exact steps for others to reproduce the error

Extract the above zip file and open append_obj_undo_crash_from_template_icon_view.blend
Run the script
In the 3d view sidebar, a new panel 'Append Object' appears under the Tool tab
In the Append Object panel, click on the thumbnail, and again on the blender icon inside

the append object op was called and a cube was appended to the scene

Undo
- Blender should crash!!!
Start Blender again and run the script again
This time, run the append op using the button below the thumbnail, then undo again
- Blender should not crash this time

**System Information** Operating system: Linux-4.15.0-112-generic-x86_64-with-debian-buster-sid 64 Bits Graphics card: GeForce GTX 1050/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 435.21 **Blender Version** Broken: version: 2.83.5, branch: master, commit date: 2020-08-19 06:07, hash: `c2b144df39`, 2.90 is also affected **Short description of error** I have an operator that appends an object from another file. Calling it and undoing right after, works as expected unless that same operator is called from an update function of an enum, storing preview collection items, as is done for template icon views. Blender will crash on 2.83.5 and 2.90. [append_obj_undo_crash_from_template_icon_view.zip](https://archive.blender.org/developer/F8818731/append_obj_undo_crash_from_template_icon_view.zip) [video demo ](https://www.youtube.com/watch?v=NUhugExZu1g) **Exact steps for others to reproduce the error** * Extract the above zip file and open append_obj_undo_crash_from_template_icon_view.blend * Run the script * In the 3d view sidebar, a new panel 'Append Object' appears under the Tool tab * In the Append Object panel, click on the thumbnail, and again on the blender icon inside - the append object op was called and a cube was appended to the scene * Undo - Blender should crash!!! * Start Blender again and run the script again * This time, run the append op using the button below the thumbnail, then undo again - Blender should not crash this time

MACHIN3 commented

2020-08-29 13:22:46 +02:00

Added subscriber: @MACHIN3

Piotr commented

2020-08-29 13:29:34 +02:00

Added subscriber: @Pinnhead

Piotr commented

2020-08-29 13:29:34 +02:00

Exactly the same behaviour in

System Information
Operating system: Windows-10-10.0.19041-SP0 64 Bits
Graphics card: GeForce GTX 1080/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 452.06

Blender Version
Broken: version: 2.90.0 Beta, branch: master, commit date: 2020-08-25 16:00, hash: 21cb6f09ff

Exactly the same behaviour in **System Information** Operating system: Windows-10-10.0.19041-SP0 64 Bits Graphics card: GeForce GTX 1080/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 452.06 **Blender Version** Broken: version: 2.90.0 Beta, branch: master, commit date: 2020-08-25 16:00, hash: `21cb6f09ff`

Joao Paulo commented

2020-08-29 14:38:25 +02:00

Added subscriber: @gendosplace

Joao Paulo commented

2020-08-29 14:38:25 +02:00

Confirm crash.

System Information
Operating system: Windows-10-10.0.19041-SP0 64 Bits
Graphics card: GeForce RTX 2070 Super / NVIDIA Corporation 4.5.0 NVIDIA 452.06

Blender Version
2.83.5, branch: master, commit date: 2020-08-19 06:07, hash: c2b144df39

Confirm crash. **System Information** Operating system: Windows-10-10.0.19041-SP0 64 Bits Graphics card: GeForce RTX 2070 Super / NVIDIA Corporation 4.5.0 NVIDIA 452.06 **Blender Version** 2.83.5, branch: master, commit date: 2020-08-19 06:07, hash: c2b144df39

MACHIN3 commented

2020-08-31 11:25:12 +02:00

Added subscriber: @rjg

MACHIN3 commented

2020-08-31 11:25:12 +02:00

@rjg Shouldn't this be BF Blender (2.90) too? It's confirmed two times to crash 2.90.

Robert Guetzkow commented

2020-08-31 11:52:12 +02:00

@MACHIN3 Phabricator doesn't let me tag more than one version. I can add both tags in the UI but one is discarded when I save the edit. Therefore, I tagged to lowest version since any fix for 2.83 will always have to be checked for inclusion into the current release as well.

MACHIN3 commented

2020-08-31 12:42:15 +02:00

Understood, thanks!

Robert Guetzkow commented

2020-08-31 23:38:44 +02:00

Changed status from 'Needs Triage' to: 'Confirmed'

Robert Guetzkow commented

2020-08-31 23:38:44 +02:00

This can be easily reproduced with the provided instructions. base->object in BKE_scene_object_base_flag_sync_from_base was NULL.

BKE_scene_object_base_flag_sync_from_base(Base * base) Line 1779	C
BKE_scene_set_background(Main * bmain, Scene * scene) Line 990	C
setup_app_data(bContext * C, BlendFileData * bfd, const unsigned char * filepath, const BlendFileReadParams * params, ReportList * reports) Line 384	C
setup_app_blend_file_data(bContext * C, BlendFileData * bfd, const unsigned char * filepath, const BlendFileReadParams * params, ReportList * reports) Line 414	C
BKE_blendfile_read_from_memfile(bContext * C, MemFile * memfile, const BlendFileReadParams * params, ReportList * reports) Line 511	C
BKE_memfile_undo_decode(MemFileUndoData * mfu, const int undo_direction, const bool use_old_bmain_data, bContext * C) Line 87	C
memfile_undosys_step_decode(bContext * C, Main * bmain, UndoStep * us_p, int undo_direction, bool UNUSED_is_final) Line 192	C
undosys_step_decode(bContext * C, Main * bmain, UndoStack * ustack, UndoStep * us, int dir, bool is_final) Line 211	C
BKE_undosys_step_undo_with_data_ex(UndoStack * ustack, bContext * C, UndoStep * us, bool use_skip) Line 695	C
BKE_undosys_step_undo_with_data(UndoStack * ustack, bContext * C, UndoStep * us) Line 723	C
BKE_undosys_step_undo(UndoStack * ustack, bContext * C) Line 728	C
ed_undo_step_impl(bContext * C, int step, const unsigned char * undoname, int undo_index, ReportList * reports) Line 208	C
ed_undo_step_direction(bContext * C, int step, ReportList * reports) Line 272	C
ed_undo_exec(bContext * C, wmOperator * op) Line 407	C
wm_operator_invoke(bContext * C, wmOperatorType * ot, wmEvent * event, PointerRNA * properties, ReportList * reports, const bool poll_only, bool use_last_properties) Line 1312	C
wm_handler_operator_call(bContext * C, ListBase * handlers, wmEventHandler * handler_base, wmEvent * event, PointerRNA * properties, const unsigned char * kmi_idname) Line 2131	C
wm_handlers_do_keymap_with_keymap_handler(bContext * C, wmEvent * event, ListBase * handlers, wmEventHandler_Keymap * handler, wmKeyMap * keymap, const bool do_debug_handler) Line 2441	C
wm_handlers_do_intern(bContext * C, wmEvent * event, ListBase * handlers) Line 2738	C
wm_handlers_do(bContext * C, wmEvent * event, ListBase * handlers) Line 2865	C
wm_event_do_handlers(bContext * C) Line 3398	C
WM_main(bContext * C) Line 485	C
main(int argc, const unsigned char * * UNUSED_argv_c) Line 548	C

This can be easily reproduced with the provided instructions. `base->object` in `BKE_scene_object_base_flag_sync_from_base` was `NULL`. ``` BKE_scene_object_base_flag_sync_from_base(Base * base) Line 1779 C BKE_scene_set_background(Main * bmain, Scene * scene) Line 990 C setup_app_data(bContext * C, BlendFileData * bfd, const unsigned char * filepath, const BlendFileReadParams * params, ReportList * reports) Line 384 C setup_app_blend_file_data(bContext * C, BlendFileData * bfd, const unsigned char * filepath, const BlendFileReadParams * params, ReportList * reports) Line 414 C BKE_blendfile_read_from_memfile(bContext * C, MemFile * memfile, const BlendFileReadParams * params, ReportList * reports) Line 511 C BKE_memfile_undo_decode(MemFileUndoData * mfu, const int undo_direction, const bool use_old_bmain_data, bContext * C) Line 87 C memfile_undosys_step_decode(bContext * C, Main * bmain, UndoStep * us_p, int undo_direction, bool UNUSED_is_final) Line 192 C undosys_step_decode(bContext * C, Main * bmain, UndoStack * ustack, UndoStep * us, int dir, bool is_final) Line 211 C BKE_undosys_step_undo_with_data_ex(UndoStack * ustack, bContext * C, UndoStep * us, bool use_skip) Line 695 C BKE_undosys_step_undo_with_data(UndoStack * ustack, bContext * C, UndoStep * us) Line 723 C BKE_undosys_step_undo(UndoStack * ustack, bContext * C) Line 728 C ed_undo_step_impl(bContext * C, int step, const unsigned char * undoname, int undo_index, ReportList * reports) Line 208 C ed_undo_step_direction(bContext * C, int step, ReportList * reports) Line 272 C ed_undo_exec(bContext * C, wmOperator * op) Line 407 C wm_operator_invoke(bContext * C, wmOperatorType * ot, wmEvent * event, PointerRNA * properties, ReportList * reports, const bool poll_only, bool use_last_properties) Line 1312 C wm_handler_operator_call(bContext * C, ListBase * handlers, wmEventHandler * handler_base, wmEvent * event, PointerRNA * properties, const unsigned char * kmi_idname) Line 2131 C wm_handlers_do_keymap_with_keymap_handler(bContext * C, wmEvent * event, ListBase * handlers, wmEventHandler_Keymap * handler, wmKeyMap * keymap, const bool do_debug_handler) Line 2441 C wm_handlers_do_intern(bContext * C, wmEvent * event, ListBase * handlers) Line 2738 C wm_handlers_do(bContext * C, wmEvent * event, ListBase * handlers) Line 2865 C wm_event_do_handlers(bContext * C) Line 3398 C WM_main(bContext * C) Line 485 C main(int argc, const unsigned char * * UNUSED_argv_c) Line 548 C ```

Daniel Grauer commented

2020-09-04 21:36:08 +02:00

Added subscriber: @DanielGrauer

Bastien Montagne commented

2020-09-14 10:10:16 +02:00

Added subscriber: @mont29

Bastien Montagne commented

2020-09-14 10:10:16 +02:00

Changed status from 'Confirmed' to: 'Archived'

Bastien Montagne closed this issue

2020-09-14 10:10:16 +02:00

Bastien Montagne self-assigned this 2020-09-14 10:10:16 +02:00

Bastien Montagne commented

2020-09-14 10:10:16 +02:00

Answer is very simple: don't do this. Never ever call an operator from a property update function, this is begging for all kind of issues. Just don't.

Robert Guetzkow commented

2020-09-14 10:22:39 +02:00

@mont29 Perhaps this should be added to the Gotchas page, if it's a known limitation?

@mont29 Perhaps this should be added to the [Gotchas](https://docs.blender.org/api/current/info_gotcha.html#help-my-script-crashes-blender) page, if it's a known limitation?

Bastien Montagne commented

2020-09-14 10:42:14 +02:00

@rjg Good point, this page needs some updates for undo/redo section too actually.

MACHIN3 commented

2020-09-14 10:44:19 +02:00

Since this is crashing Blender, and since it never was was problem before 2.83, I feel like this warants more than "don't do this".

Bastien Montagne commented

2020-09-14 11:00:41 +02:00

New undo code comes with more complexity and responsibility (old one, by nuking and reloading the whole blend file, was safer, since it was not possible to get invalid pointers out of it, if stored undo step was OK). Calling an operator from an update callback has never been recommended though, those callbacks are supposed to remain very fast and efficient, and as localized as possible. Especially messing with the Main database has always been rather bad idea, fact that it has been working previously was mere luck, definitively not by design.

MACHIN3 commented

2020-09-14 11:16:56 +02:00

Understood, thanks for clarifying!

Thomas Dinges added this to the 2.83 LTS milestone 2023-02-08 16:35:26 +01:00

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Crash when appending object using operator called from template icon view, followed by undo #80217