Platform: Code4rena
Start Date: 01/09/2023
Pot Size: $36,500 USDC
Total HM: 4
Participants: 70
Period: 6 days
Judge: kirk-baird
Id: 281
League: ETH
Rank: 66/70
Findings: 1
Award: $7.08
🌟 Selected for report: 0
🚀 Solo Findings: 0
🌟 Selected for report: adriro
Also found by: 0x6980, 0xStalin, 0xanmol, 0xmystery, 0xpanicError, Arz, Aymen0909, BenRai, Breeje, Lalanda, MohammedRizwan, Raihan, SovaSlava, Stormreckson, Udsen, ast3ros, bin2chen, castle_chain, catellatech, codegpt, dev0cloo, gkrastenov, hals, klau5, kutugu, ladboy233, matrix_0wl, nirlin, ohm, peanuts, pipidu83, sandy, wahedtalash77
7.08 USDC - $7.08
function _burnShares( address _account, uint256 _sharesAmount ) internal whenNotPaused returns (uint256) { require(_account != address(0), "BURN_FROM_THE_ZERO_ADDRESS"); _beforeTokenTransfer(_account, address(0), _sharesAmount); uint256 accountShares = shares[_account]; require(_sharesAmount <= accountShares, "BURN_AMOUNT_EXCEEDS_BALANCE"); uint256 preRebaseTokenAmount = getRUSDYByShares(_sharesAmount); totalShares -= _sharesAmount; shares[_account] = accountShares - _sharesAmount; // ISSUE ----> = postRebaseTokenAmount is not getting the post rusdy tokens //Because _sharesAmount is not updating. uint256 postRebaseTokenAmount = getRUSDYByShares(_sharesAmount);
In the above function there is no possibility of becoming the address is 0.So we can avoid the require statement .
require(_account != address(0), "BURN_FROM_THE_ZERO_ADDRESS");
#0 - c4-pre-sort
2023-09-08T08:18:17Z
raymondfam marked the issue as low quality report
#1 - c4-judge
2023-09-21T10:06:46Z
kirk-baird marked the issue as grade-c
#2 - c4-judge
2023-09-23T00:44:49Z
kirk-baird marked the issue as grade-b