Merge pull request #9776 from vegaprotocol/9773

fix: ensure invalid market for transfer doesn't panic
vegaprotocol · Oct 13, 2023 · caae176 · caae176
2 parents fc28fea + 822fec6
commit caae176
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -464,6 +464,7 @@
 - [8702](https://github.com/vegaprotocol/vega/issues/8702) - Fix panic on auction exit after stop orders expired in auction
 - [8703](https://github.com/vegaprotocol/vega/issues/8703) - Wire stop orders cancellation
 - [8698](https://github.com/vegaprotocol/vega/issues/8698) - Always set the `ToAccount` field when clearing fees.
+- [9773](https://github.com/vegaprotocol/vega/issues/9773) - Ensure invalid market for transfer doesn't panic
 
 ## 0.71.0
 

diff --git a/core/execution/common/market_activity_tracker.go b/core/execution/common/market_activity_tracker.go
@@ -711,6 +711,9 @@ func (mat *MarketActivityTracker) calculateMetricForParty(asset, party string, m
 	// for each market in scope, for each epoch in the time window get the metric entry, sum up for each epoch in the time window and divide by window size (or calculate variance - for volatility)
 	for _, market := range markets {
 		marketTracker := assetTrackers[market]
+		if marketTracker == nil {
+			continue
+		}
 		switch metric {
 		case vega.DispatchMetric_DISPATCH_METRIC_AVERAGE_POSITION:
 			uTotal += marketTracker.getPositionMetricTotal(party, windowSize)