Submission Details
Severity:
`setMinter` and `setBurner` functions in RToken contract are useless and can be removed.
Summary
setMinter and setBurner functions in RToken contract are defined as follows:
// @audit LOW: _burner never used
function setBurner(address burner) external onlyOwner {
if (burner == address(0)) revert InvalidAddress();
_burner = burner;
emit BurnerSet(_burner);
}
// @audit LOW: _minter never used
function setMinter(address minter) external onlyOwner {
if (minter == address(0)) revert InvalidAddress();
_minter = minter;
emit MinterSet(_minter);
}
The issue arises because these 2 functions are supposed to set a minter and burner addresses, using _minter and _burner storage variables. But these 2 variables are never used in the code.
Impact
The impact of this issue is low.
Tools Used
Manual review.
Recommendations
Make sure to remove setMinter and setBurner functions from the RToken contract.
Prize pool breakdown
Total prize
77,280 USDC
nSLOC
3,86420
USDC / LOC
74,000
3,280
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.