Ticket #8384 (closed: fixed)
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: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