Fix a few "Uninitialized scalar field" warnings #4777

pull laanwj wants to merge 1 commits into bitcoin:master from laanwj:2014_08_uninit_scalar_field changing 6 files +11 −7

laanwj commented at 1:31 PM on August 28, 2014: member

Fix a few warnings reported by Coverity. None of these is critical, but making sure that class fields are initialized can avoid heisenbugs.

Fix a few "Uninitialized scalar field" warnings

Fix a few warnings reported by Coverity.
None of these is critical, but making sure that class fields are
initialized can avoid heisenbugs.

8bdd2877c4

jgarzik commented at 1:33 PM on August 28, 2014: contributor

ut ACK
BitcoinPullTester commented at 1:45 PM on August 28, 2014: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4777_8bdd2877c4f959b0c4b93f3d9d1f465fb4960f3f/ for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.
laanwj added the label Improvement on Aug 28, 2014
sipa commented at 9:17 PM on August 28, 2014: member

ut ACK
Diapolo commented at 9:24 PM on August 28, 2014: none

ut ACK Why did your compiler warn about these now but not before?
sipa commented at 10:07 PM on August 28, 2014: member

Not his compiler, but Coverty, a static analysis tool for finding bugs in software.
laanwj merged this on Aug 29, 2014
laanwj closed this on Aug 29, 2014
laanwj referenced this in commit faeb341da3 on Aug 29, 2014
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels