Page MenuHome

Crash picking object from eyedropper
Closed, ResolvedPublic

Description

With the default blend file:

  • Duplicate the default cube.
  • Add a boolean modifier.
  • Use the eye-dropper to pick an object.

Event Timeline

Campbell Barton (campbellbarton) renamed this task from Crash picking objects to Crash picking object from eyedropper.Thu, Oct 10, 5:35 AM

Possible fix, however I think it'd be better to avoid accessing the button after the operator runs since running an operator could do anything including freeing the button.

diff --git a/source/blender/editors/interface/interface_handlers.c b/source/blender/editors/interface/interface_handlers.c
index 174c2710863..f73403a47df 100644
--- a/source/blender/editors/interface/interface_handlers.c
+++ b/source/blender/editors/interface/interface_handlers.c
@@ -3968,7 +3968,9 @@ static bool ui_do_but_extra_operator_icon(bContext *C,

   if (op_icon) {
     ui_but_extra_operator_icon_apply(C, but, op_icon);
-    button_activate_exit(C, but, data, false, false);
+    if (but->active == data) {
+      button_activate_exit(C, but, data, false, false);
+    }
     return true;
   }
Campbell Barton (campbellbarton) lowered the priority of this task from Needs Triage by Developer to Confirmed, Medium.