Submission Details
Severity:
The `ChristimasDinner:_refundERC20` NatSpec Assumes No Re-Entrancy, but Re-Entrancy Is Possible
Impact
The NatSpec documentation is incorrect, which could mislead developers or auditors reading the code.
Tools Used
Foundry
Recommendations
Remove the incorect natspec line.
/**
- * @dev ERC20 withdrawal of all user funds. No concern for Reentrancy
+ * @dev ERC20 withdrawal of all user funds.
* since refund() uses a Mutex Lock
* @param _to payable address passed from refund()
*/
function _refundERC20(address _to) internal {
i_WETH.safeTransfer(_to, balances[_to][address(i_WETH)]);
.
.
}
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.