Bevel tool crashes #62018

New Issue

William Reynish · 2019-02-27T20:14:11+01:00

William Reynish commented

2019-02-27 20:14:11 +01:00

System Information
Operating system: macOS 10.14.2
Graphics card: Intel Iris Plus Graphics 640 1536 MB

Blender Version
Broken: 2.80, 2019-02-27

Short description of error
Using the bevel tool crashes every time

Exact steps for others to reproduce the error

Open default scene
Enter Edit Mode
Switch to the Bevel tool in the toolbar
Click and drag in the viewport to bevel
Crash

**System Information** Operating system: macOS 10.14.2 Graphics card: Intel Iris Plus Graphics 640 1536 MB **Blender Version** Broken: 2.80, 2019-02-27 **Short description of error** Using the bevel tool crashes every time **Exact steps for others to reproduce the error** - Open default scene - Enter Edit Mode - Switch to the Bevel tool in the toolbar - Click and drag in the viewport to bevel - Crash

William Reynish commented

2019-02-27 20:14:11 +01:00

Added subscriber: @WilliamReynish

Howard Trickey was assigned by William Reynish

2019-02-27 20:14:24 +01:00

William Reynish commented

2019-02-27 20:16:09 +01:00

Same for Inset Faces too - crashes every time. Could be the same issue.

Martin Capitanio commented

2019-02-27 20:35:48 +01:00

Added subscriber: @capnm

Martin Capitanio commented

2019-02-27 20:35:48 +01:00

With the inset op:

known worked b80ff7e2f1
broken 3ad2d6caef

Thread 1 "blender8debug" received signal SIGSEGV, Segmentation fault.
0x0000555557c282c1 in initTransInfo (C=0x7fffcf06d488, t=0x7fffadbf9808, op=0x0, event=0x0)
    at source/blender/editors/transform/transform_generics.c:1525
1525		if ((prop = RNA_struct_find_property(op->ptr, "orient_axis"))) {
(gdb) l
1520				t->view = NULL;
1521			}
1522			t->around = V3D_AROUND_CENTER_BOUNDS;
1523		}
1524	
=> 1525		if ((prop = RNA_struct_find_property(op->ptr, "orient_axis"))) {
1526			t->orient_axis = RNA_property_enum_get(op->ptr, prop);
1527		}
1528		if ((prop = RNA_struct_find_property(op->ptr, "orient_axis_ortho"))) {
1529			t->orient_axis_ortho = RNA_property_enum_get(op->ptr, prop);

backtrace.txt

With the **inset** op: known worked b80ff7e2f1 broken 3ad2d6caef ``` Thread 1 "blender8debug" received signal SIGSEGV, Segmentation fault. 0x0000555557c282c1 in initTransInfo (C=0x7fffcf06d488, t=0x7fffadbf9808, op=0x0, event=0x0) at source/blender/editors/transform/transform_generics.c:1525 1525 if ((prop = RNA_struct_find_property(op->ptr, "orient_axis"))) { (gdb) l 1520 t->view = NULL; 1521 } 1522 t->around = V3D_AROUND_CENTER_BOUNDS; 1523 } 1524 => 1525 if ((prop = RNA_struct_find_property(op->ptr, "orient_axis"))) { 1526 t->orient_axis = RNA_property_enum_get(op->ptr, prop); 1527 } 1528 if ((prop = RNA_struct_find_property(op->ptr, "orient_axis_ortho"))) { 1529 t->orient_axis_ortho = RNA_property_enum_get(op->ptr, prop); ``` [backtrace.txt](https://archive.blender.org/developer/F6725279/backtrace.txt)

Howard Trickey removed their assignment 2019-02-27 20:39:53 +01:00

Campbell Barton was assigned by Howard Trickey

2019-02-27 20:39:53 +01:00

Howard Trickey commented

2019-02-27 20:39:53 +01:00

Added subscriber: @howardt

Howard Trickey commented

2019-02-27 20:39:53 +01:00

This is caused by the recent commit by Campbell, 8a432c1a4 , which put a lookup using op->ptr into initTransInfo. But that function is sometimes called with a null op (inset and bevel do it to find the geometric center to draw the UI dotted line from). Not sure if the right fix is to pass op down through the stack from those two places, or to protect the newly added code by "if (op)", so asking Campbell to figure out the right fix. If this is urgent to fix, we could put in the "if (op)" tests now until Campbell can get to it.

This is caused by the recent commit by Campbell, 8a432c1a4 , which put a lookup using op->ptr into initTransInfo. But that function is sometimes called with a null op (inset and bevel do it to find the geometric center to draw the UI dotted line from). Not sure if the right fix is to pass op down through the stack from those two places, or to protect the newly added code by "if (op)", so asking Campbell to figure out the right fix. If this is urgent to fix, we could put in the "if (op)" tests now until Campbell can get to it.

Campbell Barton commented

2019-02-28 02:32:52 +01:00

Changed status from 'Open' to: 'Resolved'

Campbell Barton closed this issue

2019-02-28 02:32:52 +01:00

Campbell Barton commented

2019-02-28 02:32:52 +01:00

Fixed 7188250523

Fixed 7188250523

Sign in to join this conversation.