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.

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

Fix Types.shrink() #265

Merged
merged 5 commits into from
Oct 2, 2023
Merged

Fix Types.shrink() #265

merged 5 commits into from
Oct 2, 2023

Conversation

mtabasco
Copy link
Collaborator

The function should ensure that the input number is strictly less than the product of the maximum uint64 value and a specific constant. This constant, which is 10,000,000, can be represented as 10e7.

@mtabasco mtabasco merged commit c8679b3 into jato-v2 Oct 2, 2023
4 checks passed
@mtabasco mtabasco deleted the im-overflow-fix branch October 2, 2023 11:05
liorrutenberg pushed a commit that referenced this pull request Nov 8, 2023
* Remove registerAuth implementation (#257)

* remove registerAuth implementation

* remove test

* remove all references to registerAuth in tests

* remove pending registerAuth

* deploy stage-dev 30082023

* deploy stage v1.0.0.rc4

* add bug bounty details in README

* Fix network withdrawals (#262)

* fix sequential network withdrawals issue

* A validator can voluntarily exit (#263)

* A validator can voluntarily exit

* Update local-dev.md (#260)

* Update local-dev.md

* Fix Types.shrink()  (#265)

* fix Types.shrink

* v1.0.0.rc5

* goerli_dev deployment

* Remove validator gas costs metrics (#266)

* holesky dev (#267)

* Get Network total validators count (#270)

* add SSVViews.getNetworkValidatorsCount()

* testnet holesky (#268)

* change input modifiers

* rename __gap (#269)

* OperatorLib gas optimizations (#273)

* remove named return params (#272)

* add msg.sender to ValidatorExited event (#271)

* remove storage checks

* add audit reports

---------

Co-authored-by: andrew-blox <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants