fix: add non-zero check of nextTokenID.Id for genesis

CHANGELOG.md

@@ -53,6 +53,7 @@ Ref: https://keepachangelog.com/en/1.0.0/
 ### Bug Fixes
 * chore(deps) [\#1141](https://github.com/Finschia/finschia-sdk/pull/1141) Bump github.com/cosmos/ledger-cosmos-go from 0.12.2 to 0.13.2 to fix ledger signing issue
 * (x/auth, x/slashing) [\#1179](https://github.com/Finschia/finschia-sdk/pull/1179) modify missing changes of converting to tendermint
+* (x/collection) [\#1276](https://github.com/Finschia/finschia-sdk/pull/1276) eliminates potential risk for Insufficient Sanity Check of tokenID in Genesis 
 
 ### Removed
 

x/collection/keeper/genesis.go

@@ -97,6 +97,9 @@ func (k Keeper) InitGenesis(ctx sdk.Context, data *collection.GenesisState) {
 		contractID := contractNextTokenIDs.ContractId
 
 		for _, nextTokenID := range contractNextTokenIDs.TokenIds {
+			if nextTokenID.Id.IsZero() {
+				panic("nextTokenID.Id is not supposed to be zero")
+			}
 			k.setNextTokenID(ctx, contractID, nextTokenID.ClassId, nextTokenID.Id)
 		}
 

x/collection/keeper/genesis_test.go

@@ -1,6 +1,7 @@
 package keeper_test
 
 import (
+	"github.com/Finschia/finschia-sdk/types"
 	"github.com/Finschia/finschia-sdk/x/collection"
 )
 
@@ -28,3 +29,17 @@ func (s *KeeperTestSuite) TestImportExportGenesis() {
 	newGenesis := s.keeper.ExportGenesis(s.ctx)
 	s.Require().Equal(genesis, newGenesis)
 }
+
+func (s *KeeperTestSuite) TestShouldPanicWhenZeroNextTokenIdInGenesis() {
+	dontCare := "12345678"
+	state := &collection.GenesisState{
+		Params: collection.Params{},
+		NextTokenIds: []collection.ContractNextTokenIDs{
+			{ContractId: dontCare, TokenIds: []collection.NextTokenID{
+				{ClassId: dontCare, Id: types.NewUint(0)},
+			}},
+		},
+	}
+
+	s.Require().Panics(func() { s.keeper.InitGenesis(s.ctx, state) })
+}