Skip to content

Fix: total supply overflow #134

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
Open
ViacheslavKhovrov wants to merge 2 commits into
base: md/target-shanghai
Choose a base branch
from
Open

Fix: total supply overflow #134

ViacheslavKhovrov wants to merge 2 commits into from

Conversation

@ViacheslavKhovrov
Copy link

@ViacheslavKhovrov ViacheslavKhovrov commented Jul 20, 2023

Add overflow check for total supply.

@ViacheslavKhovrov
Copy link
Author

ViacheslavKhovrov commented Jul 26, 2023 •
edited
Loading

There are two APPROVE() macros. Removed the old incorrect macro.

@MathisGD
Copy link
Collaborator

MathisGD commented Aug 13, 2023

There are two APPROVE() macros. Removed the old incorrect macro.

I opened an issue on huff-rs for this.

@MathisGD MathisGD mentioned this pull request Oct 14, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MathisGD MathisGD MathisGD approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ViacheslavKhovrov @MathisGD