Ticket #9946 (closed: fixed)
Coverity - High Impact Outstanding issues in Kernel files
Reported by: | Michael Reuter | Owned by: | Martyn Gigg |
---|---|---|---|
Priority: | critical | Milestone: | Release 3.3 |
Component: | Framework | Keywords: | Maintenance |
Cc: | Blocked By: | ||
Blocking: | Tester: | Anders Markvardsen |
Description
There are 4 more Coverity high impact outstanding issues in the Kernel module that you don't already have assigned to you.
Change History
comment:3 Changed 6 years ago by Martyn Gigg
- Status changed from assigned to inprogress
Fix uninitialized variable in DateValidator
Picked up by coverity. Refs #9946
Changeset: 2aeab171d336e810a12289e68dc7b3e801ebb29a
comment:4 Changed 6 years ago by Martyn Gigg
Add unit test for Matrix transpose.
Coverity detected an issue that looks like a false-positive but there was no unit test for it. Valgrind says there isn't and it looks like it is just an artifact of how the code/memory is Matrix is structured. Refs #9946
Changeset: fc973908a143fa324381ee337fa047c25d80209d
comment:5 Changed 6 years ago by Martyn Gigg
Fix coverity warning about thread-safe access in DataService
Refs #9946
Changeset: 71082f74eef0c5ec41e683c3d60c331defac7586
comment:6 Changed 6 years ago by Martyn Gigg
Style clean up in Kernel::DataService.
- remove any mention of workspace
- use camelCase naming rather than underscore naming
Refs #9946
Changeset: 3f886ae4418eb79e0686c4395b67939fc39d3d8b
comment:7 Changed 6 years ago by Martyn Gigg
- Status changed from inprogress to verify
- Resolution set to fixed
Branch: bugfix/9946_coverity_issues_kernel
Tester: Coverity on runs on master so as long as all tests are working then the only option here is a code review. All coverity issues assigned to me can be viewed here: https://scan6.coverity.com:8443/reports.htm#v23037/p10083/. This ticket only covers those in Kernel.
comment:8 Changed 6 years ago by Anders Markvardsen
- Status changed from verify to verifying
- Tester set to Anders Markvardsen
comment:9 Changed 6 years ago by Anders Markvardsen
- Status changed from verifying to closed
Merge remote-tracking branch 'origin/bugfix/9946_coverity_issues_kernel'
Full changeset: 3b05a438e58ac0ae1fe961e630b18ed72ce32fdf
comment:10 Changed 6 years ago by Anders Markvardsen
looks good to me plus addition style clean up
comment:11 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 10788