Submission Details
Severity:
StETHMock::_transfer is a dead function
Summary
StETHMock::_transfer is a dead function because is internal and has no callers.
Vulnerability Details
function _transfer(address _sender, address _recipient, uint256 _amount) internal override {
uint256 _sharesToTransfer = getSharesByPooledEth(_amount);
_transferShares(_sender, _recipient, _sharesToTransfer);
}
_transfer is an internal function that doesnt have any callers
Impact
Having dead code in the contract’s runtime bytecode, increase its size for no benefit.
Tools Used
Manual review
Recommendations
Remove dead/unreachable code.
Prize pool breakdown
Total prize
22,500 USDC
nSLOC
1,05421 USDC / LOC
20,000 USDC
2,500 USDC
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.