Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

core-bpf: special-case core bpf program error codes #151

Merged

Conversation

buffalojoec
Copy link
Collaborator

@buffalojoec buffalojoec commented Nov 7, 2024

This change special-cases a few error codes for the Address Lookup Table and Config programs, which otherwise would produce known mismatches between BPF and builtin.

For more information, see the comments above the change, or solana-program/config#21.

@buffalojoec
Copy link
Collaborator Author

buffalojoec commented Nov 8, 2024

@mjain-jump feel free to review, but don't merge until we land on solana-program/config#21 and solana-program/address-lookup-table#43 please.

@buffalojoec buffalojoec changed the title core-bpf: special-case config program error codes core-bpf: special-case core bpf program error codes Nov 8, 2024
@buffalojoec buffalojoec force-pushed the special-cased-codes-config branch from 59aa231 to 80205c3 Compare November 8, 2024 09:37
Copy link
Contributor

@mjain-jump mjain-jump left a comment

Choose a reason for hiding this comment

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

looks good!

@buffalojoec buffalojoec force-pushed the special-cased-codes-config branch from 80205c3 to 9f33632 Compare November 11, 2024 13:13
@buffalojoec
Copy link
Collaborator Author

@mjain-jump OK, those two linked PRs landed. PR is unchanged except for the removal of one code for each program. The programs only deal with read-only checks, not executable.

@buffalojoec buffalojoec merged commit deb4eaf into firedancer-io:agave-v2.1.0 Nov 12, 2024
1 check passed
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.

2 participants