Skip to content

fix: validate stress event amounts - #41

Draft
pushkart2 wants to merge 1 commit into
mainfrom
agent/validate-stress-events
Draft

fix: validate stress event amounts#41
pushkart2 wants to merge 1 commit into
mainfrom
agent/validate-stress-events

Conversation

@pushkart2

Copy link
Copy Markdown

What changed

  • reject non-numeric, NaN, non-positive, and oversized stress event amounts
  • apply the same boundary to gain and relief events

Why

Network event arguments were used directly in metadata arithmetic. A client could send negative or non-finite values to invert the operation or poison stress state.

Validation

  • git diff --check
  • direct Lua control-flow and boundary review

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.

1 participant