Submission Details
Severity:
Use `safeTransfer` instead of `sendValue`
Summary
Avoid using sendValue, which can lead to unpredictable results.
Vulnerability Details
sendValue is a low-level call that does not throw an exception when attempting to send token to the destination address, but instead returns a Boolean value indicating whether the delivery was successful. This means that if the send fails, it does not abort the current transaction, which can lead to some unpredictable results.
Impact
May result in loss of player funds.
Tools Used
Manual Review
Recommendations
Use safeTransfer instead of sendValue
Updates
Lead Judging Commences
Hamiltonite about 2 years ago
Submission Judgement Published Reason: User experience and design improvement
Rewards breakdown
nSLOC
143This contest has a flat reward structure with the following payouts:
100 EXP
20 EXP
2 EXP
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.