Ticket #10710 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Address coverity issue 1256945 - uninitialized scalar variable

Reported by: Harry Jeffery Owned by: Harry Jeffery
Priority: major Milestone: Release 3.3
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Pete Peterson

Description

It's an uninitialized scalar variable in code I wrote. It's an easy fix, and Coverity's flagged its impact as high, so it may as well be taken care of.

Change History

comment:1 Changed 6 years ago by Harry Jeffery

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Harry Jeffery

  • Status changed from assigned to inprogress

comment:3 Changed 6 years ago by Harry Jeffery

Refs #10710 Fix uninitialized scalar variable

Changeset: 7f91875fa5574c3531f2293ab73cc8c1029c5928

comment:4 Changed 6 years ago by Harry Jeffery

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

Testing This is a trivial change. Code inspection + verifying build has not been broken should be sufficient.

comment:5 Changed 6 years ago by Harry Jeffery

  • Priority changed from critical to major

comment:6 Changed 6 years ago by Harry Jeffery

This is being verified as pull request #92.

comment:7 Changed 6 years ago by Martyn Gigg

Please include a link back to the original trac issue, as is done in, for example, https://github.com/mantidproject/mantid/pull/91.

comment:8 Changed 6 years ago by Harry Jeffery

Added.

comment:9 Changed 6 years ago by Pete Peterson

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

comment:10 Changed 6 years ago by Harry Jeffery

retest this please

comment:11 Changed 6 years ago by Harry Jeffery

retest this please

comment:12 Changed 6 years ago by Martyn Gigg

retest this please

comment:13 Changed 6 years ago by Martyn Gigg

retest this please

comment:14 Changed 6 years ago by Pete Peterson

Merge pull request #92 from mantidproject/feature/10710_coverity_uninit_variable_issue

Refs #10710 Fix uninitialized scalar variable

Changeset: c9a6f7f0907efb8fa0a6a35d97a0642ee4ed597e

comment:15 Changed 6 years ago by Pete Peterson

  • Status changed from verifying to closed

Merge pull request #92 from mantidproject/feature/10710_coverity_uninit_variable_issue

Refs #10710 Fix uninitialized scalar variable

Full changeset: c9a6f7f0907efb8fa0a6a35d97a0642ee4ed597e

comment:16 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 11552

Note: See TracTickets for help on using tickets.