Skip to content

Commit

Permalink
Report for issue #432 updated by DadeKuma
Browse files Browse the repository at this point in the history
  • Loading branch information
code423n4 committed Mar 6, 2023
1 parent e0af0ca commit 9170663
Showing 1 changed file with 8 additions and 3 deletions.
11 changes: 8 additions & 3 deletions data/DadeKuma-Q.md
Original file line number Diff line number Diff line change
Expand Up @@ -168,7 +168,12 @@ File: Ethos-Core/contracts/LUSDToken.sol

If `distributionPeriod` is set to zero, it won't be possible to fund the contract as there would be a division by zero in `fund`.

Add a `require(distributionPeriod > 0, "distributionPeriod = 0")` to the setter to avoid this situation.
```solidity
File: Ethos-Core\contracts\LQTY\CommunityIssuance.sol
112: rewardPerSecond = amount.div(distributionPeriod);
```
Add a `require(distributionPeriod > 0, "distributionPeriod = 0")` to `updateDistributionPeriod` to avoid this situation.


*There is 1 instance of this issue.*
Expand Down Expand Up @@ -201,8 +206,8 @@ File: Ethos-Core/contracts/LQTY/CommunityIssuance.sol
File: Ethos-Core/contracts/LQTY/LQTYStaking.sol
135: lusdToken.transfer(msg.sender, LUSDGain);
171: lusdToken.transfer(msg.sender, LUSDGain);
135: lusdToken.transfer(msg.sender, LUSDGain);
171: lusdToken.transfer(msg.sender, LUSDGain);
```

Expand Down

0 comments on commit 9170663

Please sign in to comment.