VIS-2 / taobank-04-24

0 stars 0 forks source link

Risk of locked funds inside `VaultFactoryZapper` #32

Open DanailYordanov opened 4 months ago

DanailYordanov commented 4 months ago

Context

VaultFactoryZapper::receive()

Description

The receive function allows users to send funds to the contract. However, there is no functionality provided to withdraw mistakenly sent funds. Therefore, having this function could lead to funds being permanently locked in the contract. As a precautionary measure, it's advisable to remove the receive function altogether.