Submission Details
Severity:
function refund()
Summary
Function refund(), refund the amount back if user doesn't want to attend to the event any more, however it doesn't check for refundable balance. Also the function doesn't check if participant exists.
Vulnerability Details
function refund() external nonReentrant beforeDeadline {
address payable _to = payable(msg.sender);
_refundERC20(_to);
_refundETH(_to);
emit Refunded(msg.sender);
}
Users who ddn’t deposit could get a refund: The function allows users to request refunds without verifying if they made an actual deposit.
Impact
Financial Loss
Tools Used
Manual review
Recommendations
Check whether a user has a refundable balance and whether the participant exists.
require(participant[msg.sender], "Not a participant");
require(
balances[msg.sender][address(i_WBTC)] > 0 ||
balances[msg.sender][address(i_WETH)] > 0 ||
balances[msg.sender][address(i_USDC)] > 0 ||
etherBalance[msg.sender] > 0,
"No refundable balance"
);
Rewards breakdown
nSLOC
129This 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.