Part 2

Zaros
PerpetualsDEXFoundrySolidity
70,000 USDC
View results
Submission Details
Severity: high
Invalid

Lack of arrays check

## Summary
Lack of arrays comparison between `assets` and `dexSwapStrategyIds` in `FeeDistributionBranch::_performMultiDexSwap`
## Vulnerability Details
The function `FeeDistributionBranch::_performMultiDexSwap` lacks if the dex swap strategy Ids present in `dexSwapStrategyIds`
array correspond to the assets present in `assets` array.
## Impact
Unexpected behaviour
## Recommendations
Add a check to enforce that the length of `assets` array is greater than `dexSwapStrategyIds` array by 1.
```diff
+ require(assets.length - 1 == dexSwapStrategyIds.length, "Invalid array lengths");
```
Updates

Lead Judging Commences

inallhonesty Lead Judge 5 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.