code-423n4 / 2022-09-frax-findings

2 stars 1 forks source link

The user Can't swap their `frxETH` to `ETH` #275

Closed code423n4 closed 2 years ago

code423n4 commented 2 years ago

Lines of code

https://github.com/code-423n4/2022-09-frax/blob/main/src/frxETHMinter.sol#L37-L213

Vulnerability details

Impact

I’m so confused I didn't find any logic to withdrawing my funds ETH by transferring my frxETH

Recommended Mitigation Steps

Create logic for withdrawals (swap)

FortisFortuna commented 2 years ago

ETH 2.0 does not allow unstaking yet. When it eventually does, we will redeploy this minting contract with updated logic.

joestakey commented 2 years ago

Duplicate of #246