Merge pull request #3976 from filecoin-project/fix/state-miner-sectors

fix out-of-bounds when loading all sector infos
filecoin-project · Sep 23, 2020 · 2c945d3 · 2c945d3
2 parents 7d39542 + 1c1d23d
commit 2c945d3
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/chain/actors/builtin/miner/v0.go b/chain/actors/builtin/miner/v0.go
@@ -186,9 +186,9 @@ func (s *state0) LoadSectors(snos *bitfield.BitField) ([]*SectorOnChainInfo, err
 	if snos == nil {
 		infos := make([]*SectorOnChainInfo, 0, sectors.Length())
 		var info0 miner0.SectorOnChainInfo
-		if err := sectors.ForEach(&info0, func(i int64) error {
+		if err := sectors.ForEach(&info0, func(_ int64) error {
 			info := fromV0SectorOnChainInfo(info0)
-			infos[i] = &info
+			infos = append(infos, &info)
 			return nil
 		}); err != nil {
 			return nil, err