Ticket #11827 (closed: fixed)

Opened 5 years ago

Last modified 5 years ago

Fix coverity "null pointer dereferences" issues in Framework component

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

Description

This includes some 49 coverity issues, with ids between 1295473 and 1010263.

Change History

comment:1 Changed 5 years ago by Federico M Pouzols

  • Keywords Maintenance added

comment:2 Changed 5 years ago by Federico M Pouzols

  • Status changed from new to assigned
  • Owner set to Raquel Alvarez Banos

comment:3 Changed 5 years ago by Raquel Alvarez Banos

  • Status changed from assigned to inprogress

Re #11827 Fix coverity issues

Changeset: 618b0831caf5affff31cb74b1d9ae3c5f694c746

comment:4 Changed 5 years ago by Raquel Alvarez

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

This is being verified as ​pull request #813.

comment:5 Changed 5 years ago by Federico Montesino Pouzols

  • Status changed from verify to verifying
  • Tester set to Federico Montesino Pouzols

comment:6 Changed 5 years ago by Federico Montesino Pouzols

Changes look good. This should take care of one more bunch of coverity issues. There's only an issue with exception includes needed for Rules.cpp on rhel6 that is being fixed. After that I'll merge this in unless there are unexpected issues in the system/doc tests, regardless of the annoying DownloadFileTest.

comment:7 Changed 5 years ago by Federico Montesino Pouzols

Oops, a merge conflict has popped up against this branch.

comment:8 Changed 5 years ago by Raquel Alvarez Banos

Re #11827 Fix rhel6 compiler warning

Changeset: 19a95011988f0f941d556d38ca93187365e78884

comment:9 Changed 5 years ago by Raquel Alvarez Banos

Re #11827 Fix conflict

Changeset: b09529f6d730e19952bc93547411b2a91808772b

comment:10 Changed 5 years ago by Federico Montesino Pouzols

  • Status changed from verifying to closed

Merge pull request #813 from mantidproject/11827_Coverity_issues_null_pointer_dereferences

Coverity null pointer dereferences in Framework

Full changeset: ​77aa6e78c2e5c672d50f15040c5b298b2c262b4e

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​12665

Note: See TracTickets for help on using tickets.