Properly initialize CWallet::nTimeFirstKey #4273

pull tm314159 wants to merge 1 commits into bitcoin:master from tm314159:master changing 1 files +1 −0
  1. tm314159 commented at 6:49 PM on June 2, 2014: contributor

    Fixes valgrind warning about "conditional jump or move depends on uninitialised value(s)".

  2. Properly initialize CWallet::nTimeFirstKey 91855f279d
  3. sipa commented at 6:51 PM on June 2, 2014: member

    ACK.

    No need for a new pullreq, though. You can just do a forced push to the branch you used for the old pullreq, and it will automatically be updated.

  4. BitcoinPullTester commented at 7:18 PM on June 2, 2014: none

    Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/91855f279d27c3772b70c6e491684ed74684685e 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.

  5. jgarzik commented at 7:19 PM on June 2, 2014: contributor

    ACK

    And echo @sipa's comment. "git push --force" will update the branch of the previous pullreq.

  6. laanwj merged this on Jun 3, 2014
  7. laanwj closed this on Jun 3, 2014

  8. laanwj referenced this in commit 522a8fa377 on Jun 3, 2014
  9. MarcoFalke locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-20 03:16 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me