Submission Details
Severity:
The `SpookySwap::withdrawFees` function uses `transfer()` instead of `call()`.
Description
The function SpookySwap::withdrawFees in charge of sending the collected fees to the SpookySwap contract uses transfer() instead of call().
Impact
Depending on the selected contract owner, the use of transfer might render ETH impossible to withdraw due to the 2300 forwarded gas limitation.
Tools Used
Foundry, VsCode
Recommended mitigation
function withdrawFees() public onlyOwner {
uint256 balance = address(this).balance;
+ payable(owner()).call{value: balance}("");
emit FeeWithdrawn(owner(), balance);
}
Updates
Appeal created
bube 10 months ago
Submission Judgement Published Assigned finding tags:
Use of `transfer` instead of `call`
Rewards breakdown
nSLOC
109This 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.