Skip to content

fix: resolve integer division truncation and bounds validation issues… - #1056

Closed
edwarddavid929-png wants to merge 1 commit into
Stellar-split:mainfrom
edwarddavid929-png:batch-212-score-contract-fixes
Closed

edwarddavid929-png wants to merge 1 commit into
Stellar-split:mainfrom
edwarddavid929-png:batch-212-score-contract-fixes

Conversation

@edwarddavid929-png

Copy link
Copy Markdown
Contributor

… #230-#233

  • Fix effective_rate_bps calculation to use proper floating-point division to prevent truncation
  • Add bounds validation to set_global_payer_limit to ensure window_secs is within valid range
  • Add bounds validation to set_cancellation_cooldown to prevent overflow on u64 values
  • Add bounds validation to set_archive_after_ledgers to prevent overflow on u64 values

Prevents silent truncation of integer division results and enforces type bounds on configuration parameters.

Closes #230
Closes #231
Closes #232
Closes #233

…tellar-split#230-Stellar-split#233

- Fix effective_rate_bps calculation to use proper floating-point division to prevent truncation
- Add bounds validation to set_global_payer_limit to ensure window_secs is within valid range
- Add bounds validation to set_cancellation_cooldown to prevent overflow on u64 values
- Add bounds validation to set_archive_after_ledgers to prevent overflow on u64 values

Prevents silent truncation of integer division results and enforces type bounds on configuration parameters.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant