Ticket #8867 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Crash when workspace deleted after closing 3D plot

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: critical Milestone: Release 3.1
Component: GUI Keywords:
Cc: Blocked By:
Blocking: Tester: Karl Palmen

Description

To reproduce:

  • load MAR11060.raw from systemtests/Data (actual dataset doesn't really matter);
  • double click on the workspace to bring up the matrix;
  • click on the 3D Plot menu and select "3D Polygons";
  • after the graph shows up close just the graph;
  • delete the workspace;
  • now repeat these instructions and Mantid will crash.

Change History

comment:1 Changed 7 years ago by Martyn Gigg

  • Status changed from new to inprogress

Unsubscribe MantidMatrixFunction observers on object destruction.

Poco does not do this automatically and if they are left in the observers list then the next notification is dispatched and tries to be handled by a deleted object. Refs #8867

Changeset: 5c9a47646913e2249fc47df7dd435ab1ebdef13a

comment:2 Changed 7 years ago by Martyn Gigg

  • Status changed from inprogress to verify
  • Resolution set to fixed

Branch: bugfix/8867_crash_ws_delete_3dplot

Tester: At a minimum ensure that the steps above no longer crash Mantid. It will also be worth checking that all of the other 3D plot types in the menu don't have the same problem.

comment:3 Changed 7 years ago by Karl Palmen

  • Status changed from verify to verifying
  • Tester set to Karl Palmen

comment:4 Changed 7 years ago by Karl Palmen

Works with the MAR11001.raw file found in AutoTestData.

comment:5 Changed 7 years ago by Martyn Gigg

Unsubscribe MantidMatrixFunction observers on object destruction.

Poco does not do this automatically and if they are left in the observers list then the next notification is dispatched and tries to be handled by a deleted object. Refs #8867

Changeset: fab6d57ed5060bbabf1be89a8845e3398bd86934

comment:6 Changed 7 years ago by Karl Palmen

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/bugfix/8867_crash_ws_delete_3dplot'

Full changeset: 1b9fccadafbba5d1d43491994d896e120a452de0

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 9711

Note: See TracTickets for help on using tickets.