Submitted by Hound
Note: this finding was reported via the winning Automated Findings report. It was declared out of scope for the audit, but is being included here for completeness.
These functions lack a refund mechanism for excess Ether sent by the caller, resulting in locked funds within the contract. To rectify this, the function should be modified to implement a refund for any surplus amount.
There are 3 instances of this issue.
[233-238, 266-271, 361-364]
a2rocket (NextGen) confirmed
0xsomeone (judge) commented:
For this audit, 29 reports were submitted by wardens detailing low risk and non-critical issues. The report highlighted below by The_Kakers received the top score from the judge.
The following wardens also submitted reports: immeas, oakcobalt, DeFiHackLabs, Tadev, cheatc0d3, devival, r0ck3tz, Madalad, ZanyBonzy, dy, Arabadzhiev, Fulum, 00xSEV, lsaudit, audityourcontracts, alexfilippov314, rotcivegaf, rishabh, SpicyMeatball, MrPotatoMagic, xAriextz, mrudenko, pipidu83, tpiliposian, evmboi32, oualidpro, ZdravkoHr, and 0x3b.
