search

code-423n4 / 2022-06-putty-findings

5 stars 0 forks source link

Upgraded Q -> M from 279 [1657955636898] #467

Closed code423n4 closed 2 years ago

code423n4 commented 2 years ago

Judge has assessed an item in Issue #279 as Medium risk. The relevant finding follows:

HickupHH3 commented 2 years ago

if order.baseAsset is not weth token but you byaccdient supply msg.value > 0 then your eth is lost in the contract for ever.

Duplicate: Native ETH can be lost if it’s not utilised in exercise and fillOrder: https://github.com/code-423n4/2022-06-putty-findings/issues/226