hyperledger-labs / fabric-smart-client / 36308085104

91%

Build Type

push

github

Committed by web-flow

Commit Message

fix(vault): guard GetDirectState against nil qe and closed instances (#1892)

GetDirectState was the only accessor with a query-executor dependency
that skipped the closed-check and the write-only/nil-qe check every
sibling read path (GetState, GetStateMetadata) already applies, and it
read i.qe without the RWMutex those siblings take. A write-only
interceptor or a call racing a concurrent Reopen() could nil-deref or
race on i.qe.

Add the same IsClosed guard, RLock, and nil-qe check GetState's
FromStorage branch uses, so GetDirectState is now consistent with its
siblings.

Signed-off-by: Marcus Brandenburger <bur@zurich.ibm.com>

Coverage Stats

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

12 existing lines in 6 files now uncovered.

27752 of 32109 relevant lines covered (86.43%)

2756.13 hits per line