Ticket #9247 (closed: fixed)
Fix Coverity issues in VATES code.
Reported by: | Michael Reuter | Owned by: | Michael Reuter |
---|---|---|---|
Priority: | major | Milestone: | Release 3.2 |
Component: | GUI | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Russell Taylor |
Description
Go through the Coverity issues for the VATES code. Time boxed to two hours.
Change History
comment:3 Changed 7 years ago by Michael Reuter
Refs #9247. Fixing unchecked return on iterator.
Changeset: beabf6ab19e693f5bfaa239d34f3386f706a53f6
comment:4 Changed 7 years ago by Michael Reuter
Refs #9247. Fixing unchecked dynamic cast.
Changeset: 6526b97acb19bdf5af27d6f2d847d784bc49bc9f
comment:5 Changed 7 years ago by Michael Reuter
Refs #9247. Fixing some uninitialized members.
Changeset: 5ffad5c482256b84c9d8c996419db76ef8f3febd
comment:6 Changed 7 years ago by Michael Reuter
Refs #9247. Making RHEL6 happy.
Having to lobotomize C++11 code since RHEL6 can't cope. I hate RHEL6.
Changeset: 5663c9d6a845bffc8402eb4d652f20547784e1ec
comment:7 Changed 6 years ago by Michael Reuter
- Status changed from inprogress to verify
- Resolution set to fixed
The branch can be found here.
To test: All the VateAPI unti tests should still work. One can do quick functionality tests with the VSI. The best thing to do is look at coverity, but I've never been able to figure out how to determine what's fixed.
comment:8 Changed 6 years ago by Russell Taylor
- Status changed from verify to verifying
- Tester set to Russell Taylor
comment:9 Changed 6 years ago by Russell Taylor
- Status changed from verifying to closed
Merge remote branch 'origin/bugfix/9247_vates_coverity_issues'
Full changeset: 099ed882ca11033af6ee3c97b726c267a9a8b2cc
comment:10 Changed 6 years ago by Russell Taylor
Looks OK from inspection. As the coverity job runs on master it's unfortunately not possible to see from there whether an issue has been fixed before the ticket is closed and merged.
comment:11 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 10090