Judge has assessed an item in Issue #182 as Medium risk. The relevant finding follows:
L-Can send ETH more than buyOption premium required
Link.
require(msg.value >= premium) should be require(msg.value == premium) to prevent user send too much eth. Only beneficiary benefit from this. Contract should protect user from these nuance mistakes.
HardlyDifficult
commented
2 years ago
Judge has assessed an item in Issue #182 as Medium risk. The relevant finding follows:
L-Can send ETH more than buyOption premium required
Link. require(msg.value >= premium) should be require(msg.value == premium) to prevent user send too much eth. Only beneficiary benefit from this. Contract should protect user from these nuance mistakes.