We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
WatchPug
https://github.com/code-423n4/2021-12-amun/blob/cf890dedf2e43ec787e8e5df65726316fda134a1/contracts/basket/contracts/facets/Basket/BasketFacet.sol#L148-L148
require(!this.getLock(), "POOL_LOCKED");
https://github.com/code-423n4/2021-12-amun/blob/cf890dedf2e43ec787e8e5df65726316fda134a1/contracts/basket/contracts/facets/Basket/BasketFacet.sol#L282-L286
function getLock() external view override returns (bool) { LibBasketStorage.BasketStorage storage bs = LibBasketStorage.basketStorage(); return bs.lockBlock == 0 || bs.lockBlock >= block.number; }
getLock() is being defined and used as an external function (this.getLock()), which costs more gas than using it as an internal function.
getLock()
this.getLock()
We suggest change it to require(!getLock(), "POOL_LOCKED"); for gas saving.
require(!getLock(), "POOL_LOCKED");
The text was updated successfully, but these errors were encountered:
WatchPug issue #227
03a2b23
No branches or pull requests
Handle
WatchPug
Vulnerability details
https://github.com/code-423n4/2021-12-amun/blob/cf890dedf2e43ec787e8e5df65726316fda134a1/contracts/basket/contracts/facets/Basket/BasketFacet.sol#L148-L148
https://github.com/code-423n4/2021-12-amun/blob/cf890dedf2e43ec787e8e5df65726316fda134a1/contracts/basket/contracts/facets/Basket/BasketFacet.sol#L282-L286
getLock()
is being defined and used as an external function (this.getLock()
), which costs more gas than using it as an internal function.We suggest change it to
require(!getLock(), "POOL_LOCKED");
for gas saving.The text was updated successfully, but these errors were encountered: