Skip to content

fix: remove nonexistent vesting precompile address#1196

Closed
aglichandrap wants to merge 1 commit into
cosmos:mainfrom
aglichandrap:fix/remove-vesting-precompile
Closed

fix: remove nonexistent vesting precompile address#1196
aglichandrap wants to merge 1 commit into
cosmos:mainfrom
aglichandrap:fix/remove-vesting-precompile

Conversation

@aglichandrap

Copy link
Copy Markdown

Fix: Remove nonexistent vesting precompile address

Fixes #1192

Problem

0x0000000000000000000000000000000000000803 is included in AvailableStaticPrecompiles, but no vesting precompile implementation exists in the codebase. This causes:

  1. Default params mark a nonexistent precompile as active
  2. Calls to that address hit the missing registered precompile path

Solution

Removed VestingPrecompileAddress constant and its reference from AvailableStaticPrecompiles in x/vm/types/precompiles.go.

Changes

  • Removed VestingPrecompileAddress = "0x0000000000000000000000000000000000000803" constant
  • Removed VestingPrecompileAddress from AvailableStaticPrecompiles slice

Fixes cosmos#1192

Remove 0x...0803 from AvailableStaticPrecompiles since no vesting
precompile implementation exists in the codebase.
@aglichandrap aglichandrap requested a review from a team as a code owner May 24, 2026 12:29
@greptile-apps

greptile-apps Bot commented May 24, 2026

Copy link
Copy Markdown
Contributor

PR author is not in the allowed authors list.

@aljo242

aljo242 commented May 26, 2026

Copy link
Copy Markdown
Contributor

no param migration. any chain that initialized with default params already has 0x0000000000000000000000000000000000000803 in active precompiles state. if ValidateBasic checks enabled precompiles against AvailableStaticPrecompiles, those chains will fail param validation on upgrade. needs a migration to strip the address from existing state, or a clear explanation of why it is safe to skip.

@aljo242 aljo242 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

needs param migration for existing chains that already have the vesting precompile address in state.

@aljo242

aljo242 commented May 26, 2026

Copy link
Copy Markdown
Contributor

superseded by #1193 which is more complete (also updates genesis scripts). closing in favor of that one.

@aljo242 aljo242 closed this May 26, 2026
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.

[Bug] Remove nonexistent vesting precompile address

2 participants