Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

GenerateHashID method for decData panics for nil #164

Open
0xankit opened this issue Dec 5, 2022 · 0 comments
Open

GenerateHashID method for decData panics for nil #164

0xankit opened this issue Dec 5, 2022 · 0 comments

Comments

@0xankit
Copy link
Contributor

0xankit commented Dec 5, 2022

while comparing it with decData.ZeroValue() it panics
because value is passed through empty dec struct, and being compared with Dec from ZeroValue method of decData that creates Dec through NewDec() which uses in64

modules/schema/data/base/decData_test.go

Line 138 in 02a1abc

{"panic case with nil", fields{types.Dec{}}, baseIDs.GenerateHashID(), false},

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: adding Sanitize() method to Data interface AssetMantle/modules
1 participant
@0xankit