Skip to content

Conversation

@baroooo
Copy link

@baroooo baroooo commented Nov 18, 2025

Fixes 6.1) Insufficient Check in SystemParams by ensuring it can not be set to a value < 1e18
Also adds a new error to distinguish MinBoldInSP from MinDebt

Copy link

@philbow61 philbow61 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link

@nvtaveras nvtaveras left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes

@nvtaveras nvtaveras merged commit 19e5c6d into feat/mento-v3-dev Nov 19, 2025
8 checks passed
@nvtaveras nvtaveras deleted the fix/check-min-debt-in-systemparams branch November 19, 2025 18:05
nvtaveras added a commit that referenced this pull request Nov 25, 2025
### Description

First iteration of fixes that came up as a result of the Chainsecurity audit. Some issues are still pending and will be fixed in a future PR.

List of fixes included:
- **5.1** - Batch Manager Is Not Deleted in kickFromBatch #20 
- **5.2** - Incorrect StabilityPool Interface #23 
- **5.3** - Rebalances Are Partially Allowed During Shutdown #24 
- **5.5** - Swapping Collateral Does Not Accrue Interest #19 
- **6.1** - Insufficient Check in SystemParams #25 
- **6.2** - LiquidityStrategy Cannot Be Changed mento-protocol/mento-core#642
- **6.5** - Missing Sanity Checks #26 
- **6.6** - Redistribution Penalty Must Be Smaller Than MCR #27 
- **6.8** - Whitepaper Deviations mento-protocol/whitepaper#3
- **6.9** - minDebt Bounds Do Not Allow Configuring Low-Value Currencies Correctly #28 
- **6.10** - swapCollateralForStable Event Indistinguishable From Liquidations #29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants