Context Navigation

← Previous Ticket
Next Ticket →

Ticket #9516 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Fix newly detected ALC-related Coverity issues

Reported by:	Arturs Bekasovs	Owned by:	Arturs Bekasovs
Priority:	minor	Milestone:	Release 3.2
Component:	Muon	Keywords:	ALC
Cc:		Blocked By:
Blocking:		Tester:	Martyn Gigg

Description

https://scan6.coverity.com:8443/reports.htm#v29359/p10083/fileInstanceId=14402628&defectInstanceId=4947193&mergedDefectId=668224

https://scan6.coverity.com:8443/reports.htm#v29359/p10083/fileInstanceId=14403850&defectInstanceId=4946600&mergedDefectId=668232

Change History

comment:1 Changed 6 years ago by Arturs Bekasovs

Refs #9516. Create models in the constructor

Changeset: 9f6bd7cd79f288f0e17f9c708894ea9ce57758b4

comment:2 Changed 6 years ago by Arturs Bekasovs

Refs #9516. Use static casts instead of dynamic

We are sure those will succeed and were not checking the result anyway

Changeset: eb3bb6cd7b95ae0b9b7b88d7b514dbe72ef1d0c6

comment:3 Changed 6 years ago by Arturs Bekasovs

Tester:

Check that aforementioned issues were addressed appropriately.

comment:4 Changed 6 years ago by Nick Draper

Status changed from new to assigned

comment:5 Changed 6 years ago by Arturs Bekasovs

Status changed from assigned to verify
Resolution set to fixed

comment:6 Changed 6 years ago by Martyn Gigg

Status changed from verify to verifying
Tester set to Martyn Gigg

comment:7 Changed 6 years ago by Martyn Gigg

Status changed from verifying to closed

Merge remote-tracking branch 'origin/bugfix/9516_alc_coverity_issues'

Full changeset: ffd798e132caabca6a9792f8dfd0f2efbacd1798

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 10359

Note: See TracTickets for help on using tickets.

Download in other formats: