Skip to content

Security: Lock Forfeiture & Slashing Integrity#2958

Closed
MichaelSovereign wants to merge 98 commits intoScottcjn:mainfrom
MichaelSovereign:security-fix/lock-ledger-slashing-integrity
Closed

Security: Lock Forfeiture & Slashing Integrity#2958
MichaelSovereign wants to merge 98 commits intoScottcjn:mainfrom
MichaelSovereign:security-fix/lock-ledger-slashing-integrity

Conversation

@MichaelSovereign
Copy link
Copy Markdown
Contributor

Summary

This PR hardens the asset forfeiture (slashing) process in the module to ensure economic consistency and auditability.

Changes

  1. Atomic Slashing: Wrapped the forfeiture process in a transaction to prevent race conditions during penalty enforcement.
  2. Audit Ledger Integration: Added logic to record the removal of forfeited micro-units in the system's primary table. This ensures that the 'slashing' event is traceable and that the total circulating supply correctly reflects the removal of these assets.
  3. Input Sanitization: Implemented strict sanitization and length limits on the field to prevent database pollution or log injection.
  4. Enhanced Error Reporting: Improved error messages for state transition failures.

Closes #6460

@MichaelSovereign MichaelSovereign requested a review from Scottcjn as a code owner May 2, 2026 06:10
@github-actions github-actions Bot added BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) BCOS-L2 Beacon Certified Open Source tier BCOS-L2 (required for non-doc PRs) consensus Consensus/RIP-200 related node Node server related tests Test suite changes size/XL PR: 500+ lines labels May 2, 2026
@Scottcjn
Copy link
Copy Markdown
Owner

Scottcjn commented May 3, 2026

Closing as part of Tier 0 hard-ban cleanup — see #3074 / #3104 / #3169 for the documented incident chain. All MichaelSovereign PRs are closed unread per the Tier 0 contract. No review path; no future PRs from this account will be processed. (See feedback_michaelsovereign_tier0_2026-05-02.md.)

@Scottcjn Scottcjn closed this May 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) BCOS-L2 Beacon Certified Open Source tier BCOS-L2 (required for non-doc PRs) consensus Consensus/RIP-200 related node Node server related size/XL PR: 500+ lines tests Test suite changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants