Ticket #9675 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Crash when clicking show detectors on loaded raw file

Reported by: Samuel Jackson Owned by: Martyn Gigg
Priority: blocker Milestone: Release 3.2
Component: GUI Keywords:
Cc: Blocked By:
Blocking: Tester: Samuel Jackson

Description (last modified by Samuel Jackson) (diff)

I get a crash and the following output: 

ASSERT failure in QList<T>::operator[]: "index out of range", file /usr/include/qt4/QtCore/qlist.h, line 477
[1]    18606 abort      ./bin/MantidPlot

when loading the supplied raw file, right clicking and selecting show detectors.

This appears to be because colValues is smaller than ncols at line ~1198 in MantidUI.cpp

Attachments

TSC10076.raw​ (1.3 MB) - added by Samuel Jackson 6 years ago.
File causing the seg fault

Change History

Changed 6 years ago by Samuel Jackson

File causing the seg fault

comment:1 Changed 6 years ago by Samuel Jackson

  • Description modified (diff)
  • Summary changed from SegFault when clicking show detectors to Crash when clicking show detectors on loaded raw file

comment:2 Changed 6 years ago by Martyn Gigg

  • Owner changed from Anyone to Martyn Gigg
  • Priority changed from critical to blocker
  • Status changed from new to assigned

comment:3 Changed 6 years ago by Martyn Gigg

  • Status changed from assigned to inprogress

Fixes crash with show detectors for some files.

Put an efixed value in column values even if no detector present. Refs #9675

Changeset: aff9723832349ed65960a4536f35cd84d8306f98

comment:4 Changed 6 years ago by Martyn Gigg

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

Branch: bugfix/9675_fix_crash_showdets_raw_file

Tester: Run the steps in the description and MantidPlot will no longer crash. Try some other files and make sure they don't crash too.

comment:5 Changed 6 years ago by Samuel Jackson

  • Status changed from verify to verifying
  • Tester set to Samuel Jackson

comment:6 Changed 6 years ago by Samuel Jackson

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/bugfix/9675_fix_crash_showdets_raw_file'

Full changeset: ​5d8aea53e465647578633df90a1430934e63cdb2

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​10517

Note: See TracTickets for help on using tickets.