Closed code423n4 closed 2 years ago
https://github.com/code-423n4/2022-09-frax/blob/main/src/frxETHMinter.sol#L85
The following functions are not payable but uses msg.value - therefore the function must be payable. This can lead to undesired behavior.
frxETHMinter.sol, _submit should use payable since it uses msg.value
Manual review
add the payable
payable
Internal functions cannot be payable
Lines of code
https://github.com/code-423n4/2022-09-frax/blob/main/src/frxETHMinter.sol#L85
Vulnerability details
Impact
The following functions are not payable but uses msg.value - therefore the function must be payable. This can lead to undesired behavior.
Proof of Concept
Tools Used
Manual review
Recommended Mitigation Steps
add the
payable