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

5 stars 0 forks source link

Upgraded Q -> M from 358 [1657932904826] #461

Closed code423n4 closed 1 year ago

code423n4 commented 1 year ago

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

HickupHH3 commented 1 year ago

L03 - msg.value could be locked fillOrder() and exercise() functions should check if msg.value == 0 in case if order.baseAsset != weth. Otherwise user's ether could stuck on contract.

dup of #226