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

[Bug]: MetaMorpho #2399

Closed
Jean-Grimal opened this issue Apr 4, 2024 · 1 comment · Fixed by #2403
Closed

[Bug]: MetaMorpho #2399

Jean-Grimal opened this issue Apr 4, 2024 · 1 comment · Fixed by #2403
Labels
bug Something isn't working

Comments

@Jean-Grimal
Copy link

Describe the issue:

Slither doesn't work for MetaMorpho contract

Code example to reproduce the issue:

https://github.com/morpho-org/metamorpho

Version:

0.10.1

Relevant log output:

ERROR:SlitherSolcParsing:
Failed to generate IR for MetaMorpho.constructor. Please open an issue https://github.com/crytic/slither/issues.
MetaMorpho.constructor (src/MetaMorpho.sol#119-136):
        morpho == address(0)
        ErrorsLib.ZeroAddress()
        MORPHO = IMorpho(morpho)
        DECIMALS_OFFSET = uint8(uint256(18).zeroFloorSub(IERC20Metadata(_asset).decimals()))
        _checkTimelockBounds(initialTimelock)
        _setTimelock(initialTimelock)
        IERC20(_asset).forceApprove(morpho,type()(uint256).max)
        ERC4626(IERC20(_asset))
        ERC20Permit(_name)
        ERC20(_name,_symbol)
        Ownable(owner)
@Jean-Grimal Jean-Grimal added the bug-candidate Bugs reports that are not yet confirmed label Apr 4, 2024
@0xalpharush 0xalpharush added bug Something isn't working and removed bug-candidate Bugs reports that are not yet confirmed labels Apr 5, 2024
@0xalpharush 0xalpharush changed the title [Bug-Candidate]: MetaMorpho [Bug]: MetaMorpho Apr 5, 2024
@0xalpharush 0xalpharush mentioned this issue Apr 6, 2024
Merged
@0xalpharush 0xalpharush linked a pull request Apr 7, 2024 that will close this issue
resolve available definitions from import by reference ID #2403
Merged
@0xalpharush
Copy link
Contributor

closed by #2403

@github-actions github-actions bot mentioned this issue May 1, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

resolve available definitions from import by reference ID crytic/slither
2 participants
@Jean-Grimal @0xalpharush