Ticket #11828 (closed: fixed)

Opened 5 years ago

Last modified 5 years ago

Fix coverity "null pointer dereferences" issues in MantidPlot component

Reported by: Federico M Pouzols Owned by: Roman Tolchenov
Priority: major Milestone: Release 3.5
Component: Framework Keywords: Maintenance
Cc: Blocked By:
Blocking: #11791 Tester: Raquel Alvarez

Description

This includes some 113 issues, so feel free to work on a subset of them or split this ticket into 2 or more. Many of these issues are unchecked dynamic_cast operations.

Change History

comment:1 Changed 5 years ago by Federico M Pouzols

  • Keywords Maintenance added

comment:2 Changed 5 years ago by Roman Tolchenov

  • Status changed from new to assigned
  • Owner set to Roman Tolchenov

comment:3 Changed 5 years ago by Roman Tolchenov

  • Status changed from assigned to inprogress

Re #11828. Fixed a few issues.

Changeset: 4738a2fbcea9e1f32f40894e8b6a32a61df3d032

comment:4 Changed 5 years ago by Roman Tolchenov

Re #11828. The rest of MantidPlot and some framework.

Changeset: fd819e1179bf23f9ee4eba489a5d885923513991

comment:5 Changed 5 years ago by Roman Tolchenov

Merge remote-tracking branch 'origin/master' into 11828_coverity_null_pointer_dereferences

Conflicts:

Code/Mantid/MantidPlot/src/MatrixValuesDialog.cpp

Re #11828

Changeset: bcac8f5e44ee4dd820ea6ea93aa042671344f9c9

comment:6 Changed 5 years ago by Roman Tolchenov

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

This is being verified as ​pull request #801.

comment:7 Changed 5 years ago by Raquel Alvarez

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

comment:8 Changed 5 years ago by Roman Tolchenov

Re #11828. Fix gcc and cppcheck errors.

Changeset: 2d4aba77f0191d9f53a0b5ae6a78633d226d31bf

comment:9 Changed 5 years ago by Raquel Alvarez

  • Status changed from verifying to closed

Merge pull request #801 from mantidproject/11828_coverity_null_pointer_dereferences

Coverity null pointer dereferences in MantidPlot

Full changeset: ​0d479b34f2de63de15bfb236714832576b96c634

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​12666

Note: See TracTickets for help on using tickets.