Skip to content

fix(spammer): reject overflowing transaction mix totals - #377

Open
huklaa wants to merge 1 commit into
circlefin:mainfrom
huklaa:fix-352-tx-mix-overflow
Open

huklaa wants to merge 1 commit into
circlefin:mainfrom
huklaa:fix-352-tx-mix-overflow

Conversation

@huklaa

@huklaa huklaa commented Sep 10, 2026

Copy link
Copy Markdown

Summary

  • make "TxTypeMix" aggregate-weight handling overflow-safe
  • reject invalid "--mix" totals at the parser/configuration boundary instead of panicking
  • add focused regression coverage for overflowing totals

Fixes #352

Context

I claimed #352 first and prepared this branch shortly afterward. A later duplicate PR was closed in favor of this earlier claim, so this PR continues that original work.

Validation

  • "cargo fmt --check" — passed
  • "cargo test -p spammer" — passed (75 library tests, 10 binary tests, 0 failures)
  • "cargo clippy -p spammer --all-targets -- -D warnings" — passed with 0 warnings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant