diff --git a/libnetwork/endpoint_cnt.go b/libnetwork/endpoint_cnt.go index 702fbac8d7..c63d06abe0 100644 --- a/libnetwork/endpoint_cnt.go +++ b/libnetwork/endpoint_cnt.go @@ -113,6 +113,9 @@ func (ec *endpointCnt) updateStore() error { if store == nil { return fmt.Errorf("store not found for scope %s on endpoint count update", ec.DataScope()) } + // make a copy of count and n to avoid being overwritten by store.GetObject + count := ec.EndpointCnt() + n := ec.n for { if err := ec.n.getController().updateToStore(ec); err == nil || err != datastore.ErrKeyModified { return err @@ -120,6 +123,10 @@ func (ec *endpointCnt) updateStore() error { if err := store.GetObject(datastore.Key(ec.Key()...), ec); err != nil { return fmt.Errorf("could not update the kvobject to latest on endpoint count update: %v", err) } + ec.Lock() + ec.Count = count + ec.n = n + ec.Unlock() } } @@ -136,7 +143,9 @@ retry: if inc { ec.Count++ } else { - ec.Count-- + if ec.Count > 0 { + ec.Count-- + } } ec.Unlock()