Skip to content

fix(pool): harden get_stats utilization overflow#337

Open
naufalfx805-source wants to merge 2 commits into
TrusTrove:mainfrom
naufalfx805-source:fix-pool-get-stats-overflow
Open

fix(pool): harden get_stats utilization overflow#337
naufalfx805-source wants to merge 2 commits into
TrusTrove:mainfrom
naufalfx805-source:fix-pool-get-stats-overflow

Conversation

@naufalfx805-source

@naufalfx805-source naufalfx805-source commented Jul 17, 2026

Copy link
Copy Markdown

Closes #255

Summary

  • route get_stats utilization scaling through checked multiplication
  • return PoolError::Overflow instead of allowing raw u128 multiplication overflow
  • add regression coverage for large total_funded values that overflow when scaled to basis points

Validation

  • git diff --check
  • GitHub Actions build-and-test passed

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.

[Medium] fix(pool): harden get_stats overflow: total_funded * 10000 can wrap

1 participant