Blender Git Loki

Git Commits -> Revision 852d10b

Revision 852d10b by Philipp Oeser (blender-v2.93-release)
November 2, 2021, 11:56 (GMT)
Fix T92265: Outliner crash clicking override warning buttons

`outliner_draw_overrides_buts` uses `uiDefIconBlockBut` but doing so
without defining a function callback to actually build a block.
This will make the button go down the route of spawning a popup, but
without a menu. Crash then happens later accesing the (missing) menu in
`ui_handler_region_menu`.

So while we could dive into making this usage failsafe (carefully
checking `BUTTON_STATE_MENU_OPEN` in combination with
`uiHandleButtonData->menu` being NULL all over), but it seems much more
straightforward to just use `uiDefIconBut` (instead of
`uiDefIconBlockBut`) since this Override Warning buttons seem not to
intend spawning a menu anyways?

Maniphest Tasks: T92265

Differential Revision: https://developer.blender.org/D12917

Commit Details:

Full Hash: 852d10bd3d88e593e8537168c9a057dc371be0ff
Parent Commit: 2491bc5
Lines Changed: +14, -10

1 Modified Path:

/source/blender/editors/space_outliner/outliner_draw.c (+14, -10) (Diff)
Tehnyt: Miika HämäläinenViimeksi päivitetty: 07.11.2014 14:18MiikaH:n Sivut a.k.a. MiikaHweb | 2003-2021