Uno-Re / SSIP-SSRP-contracts

8 stars 4 forks source link

[M-01] Insufficient input validation #26

Open madMax92221 opened 8 months ago

madMax92221 commented 8 months ago

Severity

Impact: Medium, because a protocol can be broken and the code could give a false calculations

Likelihood: Medium, as it can be gamed but it needs compromised / malicious owner

Description

The _rewardMultiplier param in createRiskPool() is not constrained in any way.

Another instances where an upper constrain is missing are:

Recommendation

Set reasonable lower and upper constrains for these params.

wankhede04 commented 7 months ago

Acknowledged