Cosmos-sdk: May cause mint module panic

Created on 2 Dec 2019  Â·  3Comments  Â·  Source: cosmos/cosmos-sdk

Summary of Bug

The parameter MintDenom can be modified through the gov module, so when initiating a parameter modification proposal, the legitimacy of the modified parameter should be checked. Otherwise, once an illegal value is submitted (referring to an illegal denom here, of course, it may be due to a mistake), and the proposal is approved, which may cause panic

Version

master

Steps to Reproduce

  1. Submit MintDenom parameter modification proposal
  2. Vote on the previous step's proposal

For Admin Use

  • [ ] Not duplicate issue
  • [ ] Appropriate labels applied
  • [ ] Appropriate contributors tagged
  • [ ] Contributor assigned/self-assigned
params

Most helpful comment

All 3 comments

I recall that parameter validation is still a TODO that we need to address. cc: @alexanderbez.

dup of #3473

Was this page helpful?
0 / 5 - 0 ratings