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

Account Lock check is not checking for mutability #35682

Closed
jwalin12 opened this issue Oct 30, 2024 · 1 comment
Closed

Account Lock check is not checking for mutability #35682

jwalin12 opened this issue Oct 30, 2024 · 1 comment
Labels
community Community contribution

Comments

@jwalin12
Copy link

Problem

The TooManyAccountLocks error is currently checking the length of the AccountKeys for a SantizedMessage. However, this does not allow for transactions with more than 64 accounts, even if the transaction is locking less than 64 accounts. This is a blocking issue for many features on our app, since our transactions are quite large and use multiple LUTs.

Proposed Solution

Instead of using account_keys.len(), Change the TooManyAccountLocks error to check accountKeys the number of accounts where writable = true. Only error if the writable amount is too large.

@jwalin12 jwalin12 added the community Community contribution label Oct 30, 2024
Copy link
Contributor

This repository is no longer in use. Please re-open this issue in the agave repo: https://github.com/anza-xyz/agave

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Community contribution
Projects
None yet
Development

No branches or pull requests

1 participant