Ticket #10713 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Crash when manipulating empty 3D plots

Reported by: Federico M Pouzols Owned by: Federico M Pouzols
Priority: minor Milestone: Release 3.4
Component: GUI Keywords:
Cc: Blocked By:
Blocking: Tester: Raquel Alvarez Banos

Description (last modified by Federico M Pouzols) (diff)

Follow these steps to make Mantid crash badly:

  • Create a new matrix (File->New->Matrix) and add some values (1 is enough)
  • Create a 3D plot based on the matrix
  • Delete the matrix. The plot becomes empty (but the axes are there, as opposed to really empty plots made from an empty matrix, which are just blank).
  • If you try to open Format->Plot (or Format->Scales, etc.) for the 3D plot (that is now empty), a pop-up window will tell you: "Not available for empty 3D surface plots!" but then Mantid will crash badly.

Change History

comment:1 Changed 6 years ago by Federico M Pouzols

  • Description modified (diff)

comment:2 Changed 6 years ago by Federico M Pouzols

  • Description modified (diff)

comment:3 Changed 6 years ago by Federico M Pouzols

  • Status changed from new to assigned

comment:4 Changed 6 years ago by Federico M Pouzols

Note that if you do not add any value in the matrix, the crash will still happen.

Also, this happens with any of the 3d plots, like 3d wired frame, 3d hidden line, etc.

comment:5 Changed 6 years ago by Federico Montesino Pouzols

  • Status changed from assigned to inprogress

Prevent crashes using pointer checks and try/catch, re #10713,

Changeset: 80861664c3bc3b023cb1e236a0923605eeee6cc2

comment:6 Changed 6 years ago by Federico Montesino Pouzols

be more strict with potential null after dynamic_cast, re #10713,

Changeset: 6ec414ced74a75e2da288ca386a952b917b4d1f9

comment:7 Changed 6 years ago by Federico M Pouzols

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

To test: Check the ticket description and comment #comment:4. Repeat the steps described in the ticket description, or otherwise try to make Mantid crash here.

comment:8 Changed 6 years ago by Raquel Alvarez Banos

  • Status changed from verify to verifying
  • Tester set to Raquel Alvarez Banos

comment:9 Changed 6 years ago by Raquel Alvarez Banos

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/bugfix/10713_crash_manipulating_empty_3d_plots'

Full changeset: 91b4fb1da92c4a171f1fb4a337c0da4b7d9e1bde

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 11555

Note: See TracTickets for help on using tickets.