hyperledger-labs / fabric-smart-client / 24961088813

push

github

fix(lazy): add read lock to lazyProvider.Length (#1329)

Resolves #1324.

lazyProvider.Length read len(v.cache) without holding cacheLock,
which is a data race against any concurrent write through Get,
Update, or Delete (all of which take cacheLock.Lock). The race
trips Go's runtime detector under -race.

Adds RLock/RUnlock around the read, matching what Peek already
does for cache reads, and adds TestLengthConcurrentWithWrites that
exercises Update / Delete / Length in parallel goroutines so the
race is caught by 'go test -race ./platform/common/utils/lazy/...'.

Signed-off-by: Matt Van Horn <mvanhorn@gmail.com>

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

26544 of 39654 relevant lines covered (66.94%)

3102.16 hits per line