Ticket #8384 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Fix Coverity errors for VATES code

Reported by: Michael Reuter Owned by: Michael Reuter
Priority: major Milestone: Release 3.1
Component: GUI Keywords: Maintenance
Cc: Blocked By:
Blocking: Tester: Peter Peterson

Description

The Coverity scan has picked up some issues with the VATES code. These should be addressed, but time box it to two hours.

Change History

comment:1 Changed 7 years ago by Michael Reuter

  • Status changed from new to inprogress

comment:2 Changed 7 years ago by Michael Reuter

Refs #8384. Fixing uninitialized variables.

Changeset: 4477cdbaed707eaec495962b4d3aee01e80bf0fc

comment:3 Changed 7 years ago by Michael Reuter

Refs #8384. Fixing resource leak.

The fix is being handled by deleting the unused code.

Changeset: 4ddc34541a733c07eda090e133bd4cb5198d6584

comment:4 Changed 7 years ago by Michael Reuter

Refs #8384. Fixing resource leaks.

Changeset: df01232e0e1c1864a6e73226b4e6c4bd709e8257

comment:5 Changed 7 years ago by Michael Reuter

Refs #8384. Fixing more uninitialized variables.

Changeset: 37f05e665d22a9fe6f47469b4dabce22fa4d5362

comment:6 Changed 7 years ago by Michael Reuter

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

The branch can be found here.

To test: Someone with access to Coverity will need to look at this to ensure that the number of errors from Vates code is reduced. It's not zero since the time box ran out. :-) Whoever, takes this ticket should also do a cursory check on the VSI to make sure nothing major broke.

comment:7 Changed 7 years ago by Peter Peterson

  • Status changed from verify to verifying
  • Tester set to Peter Peterson

comment:8 Changed 7 years ago by Peter Peterson

  • Status changed from verifying to closed

Merge remote branch 'origin/bugfix/8384_vates_coverity_scan'

Full changeset: e05ccc9dedb41651802ed9e5ef7fbd94168b7dda

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 9229

Note: See TracTickets for help on using tickets.