hyperledger / fabric-x-common

83%

LAST BUILD ON BRANCH main

branch: SELECT

CHANGE BRANCH

x

• No branch selected
• asn1-message-to-sign
• asn1-unmarshal
• blk-itr-ctx
• copilot/fix-data-race-logger
• coveralls
• cryptogen-org-folder-domain
• cryptogen-recreate
• dependabot/go_modules/go_modules-9c5197dcb8
• expose-ledger-api
• fabric-lib-go-v1.1.3
• fix-acl
• fix-block-verification-policy-orderer
• fix-data-hash
• fix-fileledger-newest-empty
• fix-gpl
• grpc-logger
• guide-for-configtx.md
• hash-tls-cert
• load-config-material
• main
• operation_endpoints
• oreder_shared_config_ext
• return-not-found-blk-store
• rm-para-config-block
• sidecar-notify-reject
• unique-genesis-default
• verifier-test

Build # 26457337030

Build Type

push

github

Committed by web-flow

Commit Message

Fix newest iterator on empty file ledger (#116)

#### Type of change

- Bug fix

#### Description

When `SeekPosition_Newest` is used on an empty `FileLedger`, `Iterator`
computed `Height - 1`, underflowing to `math.MaxUint64`. This makes the
iterator request an impossible block number instead of waiting from
block zero.

- Guard newest seek when ledger height is zero
- Start empty-ledger newest iterators at block zero
- Add regression coverage for the empty-ledger case

#### Related

Companion for hyperledger/fabric-x-committer#601.

Signed-off-by: Senthilnathan <cendhu@gmail.com>

Coverage Stats

4 of 4 new or added lines in 1 file covered. (100.0%)

8 existing lines in 4 files now uncovered.

14848 of 17962 relevant lines covered (82.66%)

0.92 hits per line